[PATCH] D18321: Add support for no-jump-tables

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 28 10:51:04 PDT 2016


hans added inline comments.

================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:7957
@@ -7956,1 +7956,3 @@
 
+  //Give up if function containing SI has NoUseJumpTable Attribute set
+  //to true
----------------
Please make areJTsAllowed() take a Function* parameter and do the check there instead. That way, the code will not waste time in findJumpTables() only to decide later to not build any.


http://reviews.llvm.org/D18321





More information about the llvm-commits mailing list