[PATCH] D32564: AArch64: compress jump tables to minimum size needed to reach destinations

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 9 15:23:31 PDT 2018


t.p.northover added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64AsmPrinter.cpp:519
+      MF->getInfo<AArch64FunctionInfo>()->getJumpTableEntryPCRelSymbol(JTIdx);
+  EmitToStreamer(OutStreamer, MCInstBuilder(AArch64::ADR)
+                                  .addReg(DestReg)
----------------
evandro wrote:
> It seems to me that this instruction could be moved after the load below to promote macro fusion.
The comment above explains why this has to be first.


Repository:
  rL LLVM

https://reviews.llvm.org/D32564





More information about the llvm-commits mailing list