[PATCH] D21468: Fixed LegacyPassManager inconsistency

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 7 18:19:13 PDT 2016


mehdi_amini added a comment.

I'll need more time to figure if it is the right fix or if it is a bandaid on top of the existing complexity.


================
Comment at: lib/IR/LegacyPassManager.cpp:557
@@ -545,2 +556,3 @@
         // this is just updating existing entries.
-        LastUser[LUI->first] = P;
+        if (isLastUseUnlocked(LUI->first))
+          LastUser[LUI->first] = P;
----------------
Merge with previous if?


http://reviews.llvm.org/D21468





More information about the llvm-commits mailing list