[PATCH] D87286: AArch64: make sure jump table entries can reach entire image

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Given that most jump tables are probably compressed anyway, I don't expect any big performance impact.  But it would be nice to have some numbers.



================
Comment at: llvm/lib/Target/AArch64/AArch64AsmPrinter.cpp:856
+  case 2: LdrOpcode = AArch64::LDRHHroX; break;
+  case 4: LdrOpcode = AArch64::LDRSWroX; break;
+  default:
----------------
For 32-bit, we sign-extend?  I guess this is assuming a function can't be larger than 2GB?  (I think you could avoid that assumption by using the compression infrastructure for 32-bit words.)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87286/new/

https://reviews.llvm.org/D87286