[PATCH] D30543: [JumpThreading] Perform phi-translation in SimplifyPartiallyRedundantLoad.

Jun Bum Lim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 12:35:13 PST 2017


junbuml added a comment.

We had some discussion about improving this function in https://reviews.llvm.org/D30352. Based on that I added Renato and Daniel as reviewer.



================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:1005
+    // unordered, that should have been checked when we entered the function.
+    assert(isUnordered() && "Attempting to CSE volatile or atomic loads");
+    // If this is a load on a phi pointer, phi-translate it and search
----------------
Maybe  LI->isUnordered()  ?


https://reviews.llvm.org/D30543





More information about the llvm-commits mailing list