[PATCH] D24462: [ARM] Don't convert switches to lookup tables of pointers with ROPI/RWPI

[Eli Friedman via llvm-commits]

efriedma added inline comments.

================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:5151
@@ -5145,1 +5150,3 @@
+      if (!TTI.shouldBuildLookupTablesForConstant(Value))
+        return false;
       if (!ResultLists.count(PHI))
----------------
Should we be calling ValidLookupTableConstant here instead of calling shouldBuildLookupTablesForConstant directly?


Repository:
  rL LLVM

https://reviews.llvm.org/D24462