[PATCH] D22696: [CodeGenPrep] Skip merging empty case blocks

Manman Ren via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 9 11:41:42 PST 2016


manmanren added inline comments.


================
Comment at: test/CodeGen/X86/ragreedy-hoist-spill.ll:183
+  ; CHECK: movq %r{{.*}}, {{[0-9]+}}(%rsp)
+  ; CHECK: land.rhs485
   %cmp476 = icmp eq i8 undef, 0
----------------
The whole purpose of this testing case is to make sure that a spill is not hoisted to a hotter outer loop.

As long as that is still true with your change, it is fine.


https://reviews.llvm.org/D22696





More information about the llvm-commits mailing list