[PATCH] D39832: [ARM] Split Arm jump table branch into i12 and rs suffixed versions

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 9 12:02:58 PST 2017


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

LGTM



================
Comment at: lib/Target/ARM/ARMAsmPrinter.cpp:1687
+    // Lower and emit the instruction itself, then the jump table following it.
+    // ldr pc, target
+    MCInst TmpInst;
----------------
Looks like the comment here is a bit out of date?  We don't emit the jump table here.


Repository:
  rL LLVM

https://reviews.llvm.org/D39832





More information about the llvm-commits mailing list