[PATCH] D65857: [MC][AArch64] Restrict use of signed relocation operators on MOV[NZK]

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added a comment.

MHO: The assembler is a low enough level component that the user can be presumed to know what they're doing, regardless of linker limitations. So I would prefer not to do this. If we do anything about this, we should document the limitations of the GNU linkers somewhere.



================
Comment at: lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:965
+  bool isMovKSymbolG3() const {
+    return isMovWSymbol({AArch64MCExpr::VK_ABS_G3});
+  }
----------------
We produce MOVK with PREL_G3 when the tagged-globals target feature is enabled, so we should allow this combination at least.


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

https://reviews.llvm.org/D65857