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

[NewGVN] Parenthesise assertion condition (-Wparenthesis). 

Format an assertion message while I'm here.

llvm-svn: 291710
This commit is contained in:
Davide Italiano 2017-01-11 21:58:42 +00:00
parent 6e919df2f5
commit ff69405213
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
llvm/lib/Transforms/Scalar/NewGVN.cpp
View File

@ -1536,13 +1536,12 @@ void NewGVN::verifyMemoryCongruency() const {
if (auto *FirstMUD = dyn_cast<MemoryUseOrDef>(KV.first)) { if (auto *FirstMUD = dyn_cast<MemoryUseOrDef>(KV.first)) {
auto *SecondMUD = dyn_cast<MemoryUseOrDef>(KV.second); auto *SecondMUD = dyn_cast<MemoryUseOrDef>(KV.second);
if (FirstMUD && SecondMUD) if (FirstMUD && SecondMUD)
assert(singleReachablePHIPath(FirstMUD, SecondMUD) || assert((singleReachablePHIPath(FirstMUD, SecondMUD) ||
ValueToClass.lookup(FirstMUD->getMemoryInst()) == ValueToClass.lookup(FirstMUD->getMemoryInst()) ==
ValueToClass.lookup(SecondMUD->getMemoryInst()) && ValueToClass.lookup(SecondMUD->getMemoryInst())) &&
"The instructions for these memory operations should have " "The instructions for these memory operations should have "
"been in " "been in the same congruence class or reachable through"
"the same congruence class or reachable through a single " "a single argument phi");
"argument phi");
} else if (auto *FirstMP = dyn_cast<MemoryPhi>(KV.first)) { } else if (auto *FirstMP = dyn_cast<MemoryPhi>(KV.first)) {
// We can only sanely verify that MemoryDefs in the operand list all have // We can only sanely verify that MemoryDefs in the operand list all have