[PATCH] D11566: Fix FlattenCFG to invert test when different index is encountered, and add one more operator to invert.
hfinkel@anl.gov via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 10 00:41:58 PDT 2015
hfinkel added a subscriber: hfinkel.
================
Comment at: lib/Transforms/Utils/FlattenCFG.cpp:202
@@ -201,3 +201,3 @@
Idx = CIdx;
- else if (CIdx != Idx)
- return false;
+ else if (CIdx != Idx) {
+ bool EverChanged = false;
----------------
I think that I'm missing something here: We currently might perform this change when CIdx == Idx, and with this patch, we only ever do it when CIdx != Idx. Don't you want to do it in both cases?
http://reviews.llvm.org/D11566
More information about the llvm-commits
mailing list