maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Fold StatepointBB into checks as it's only used from an NDEBUG or ASSERT 

context fixing an unused variable warning.
This commit is contained in:
Eric Christopher 2020-07-25 18:34:02 -07:00
parent cdead4f89c
commit 18975762c1
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/lib/CodeGen/SelectionDAG/StatepointLowering.cpp
View File

@ -1087,13 +1087,12 @@ void SelectionDAGBuilder::visitGCResult(const GCResultInst &CI) {
}
void SelectionDAGBuilder::visitGCRelocate(const GCRelocateInst &Relocate) {
const BasicBlock *StatepointBB = Relocate.getStatepoint()->getParent();
#ifndef NDEBUG
// Consistency check
// We skip this check for relocates not in the same basic block as their
// statepoint. It would be too expensive to preserve validation info through
// different basic blocks.
if (StatepointBB == Relocate.getParent())
if (Relocate.getStatepoint()->getParent() == Relocate.getParent())
StatepointLowering.relocCallVisited(Relocate);
auto *Ty = Relocate.getType()->getScalarType();
@ -1117,7 +1116,8 @@ void SelectionDAGBuilder::visitGCRelocate(const GCRelocateInst &Relocate) {
auto It = DPtrMap.find(DerivedPtr);
if (It != DPtrMap.end()) {
setValue(&Relocate, It->second);
assert(Relocate.getParent() == StatepointBB && "unexpected DPtrMap entry");
assert(Relocate.getParent() == Relocate.getStatepoint()->getParent() &&
"unexpected DPtrMap entry");
return;
}