[PATCH] D35609: [LICM] Make sinkRegion and hoistRegion non-recursive

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 19 10:50:24 PDT 2017


efriedma added inline comments.


================
Comment at: lib/Transforms/Scalar/LICM.cpp:354
+                     return DT->properlyDominates(B, A);
+                   });
+
----------------
properlyDominates is not a strict weak ordering, so this has undefined behavior.


https://reviews.llvm.org/D35609





More information about the llvm-commits mailing list