[PATCH] D29092: PR31729: [GVNHoist] Don't hoist unsafe scalars at -Oz

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 24 11:05:24 PST 2017


MatzeB added inline comments.


================
Comment at: llvm/test/Transforms/GVNHoist/hoist-unsafe-pr31729.ll:3-6
+; CHECK-LABEL: @main
+; CHECK: urem
+; CHECK: urem
+; CHECK: urem
----------------
How does this test work? If this is about hoisting wouldn't we always see 3 urem instructions no matter if we did the hoisting or not?


https://reviews.llvm.org/D29092





More information about the llvm-commits mailing list