[PATCH] D12166: Assign weights to edges to jump table when lowering switch statement.

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


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

Yes, lgtm. Sorry for the delay.


================
Comment at: test/CodeGen/X86/switch-jump-table.ll:58
@@ +57,3 @@
+; CHECK2-LABEL: bar:
+; CHECK2: Successors according to CFG: BB#7(16) BB#1(96)
+; CHECK2: Successors according to CFG: BB#2(16) BB#3(16) BB#4(16) BB#5(16) BB#6(32)
----------------
This isn't really checking the probabilities though, but the block placement. Maybe that's the best we can do? :-/

Maybe use a more descriptive prefix than CHECK2? CHECK-PLACEMENT?


http://reviews.llvm.org/D12166





More information about the llvm-commits mailing list