[PATCH] D12391: Fixed a bug that edge weights are not assigned correctly when lowering switch statement.

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 26 17:26:17 PDT 2015


hans accepted this revision.
hans added a subscriber: test.
hans added a comment.
This revision is now accepted and ready to land.

lgtm with some tweaks to the test.


================
Comment at: test/CodeGen/X86/switch-edge-weight.ll:6
@@ +5,3 @@
+
+; CHECK: BB#4:
+; CHECK: Successors according to CFG: BB#1(10) BB#6(10)
----------------
Please add a CHECK-LABEL and move this into or after the @test function.

The alloca, store and load, etc. seem unnecessary. Can the test just switch on the function parameter?


http://reviews.llvm.org/D12391





More information about the llvm-commits mailing list