[PATCH] D23652: [mips] N64 static relocation model support

Joerg Sonnenberger via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 23 13:44:08 PST 2016


joerg added inline comments.


================
Comment at: lib/Target/Mips/MipsISelLowering.cpp:3693
 unsigned MipsTargetLowering::getJumpTableEncoding() const {
-  if (ABI.IsN64())
+  if (ABI.IsN64() && isPositionIndependent())
     return MachineJumpTableInfo::EK_GPRel64BlockAddress;
----------------
Please leave this alone. The trade-off is intentional and should stay. If anything, it should be further refined to default to EK_GPRel32BlockAddress, but it should certainly not use absolute labels.


https://reviews.llvm.org/D23652





More information about the llvm-commits mailing list