[PATCH] D155732: [MC][COFF][AArch64] Avoid incorrect IMAGE_REL_ARM64_BRANCH26 relocations.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/lib/MC/WinCOFFObjectWriter.cpp:935
+              Offset < Layout.getSectionAddressSize(&A.getSection()))) {
+          report_fatal_error(
+              "cannot add a symbol that's outside of the section");
----------------
`Asm.getContext().reportError()`, like code earlier in this function.


================
Comment at: llvm/lib/MC/WinCOFFObjectWriter.cpp:944
+      Reloc.Symb = S;
+      FixedValue = 0;  // Zero offset
+    } else {
----------------
I don't understand why it's safe to throw away the "FixedValue" here; there could be an explicit offset.


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

https://reviews.llvm.org/D155732