[PATCH] D113220: [X86] Selective relocation relaxation for +tagged-globals

[Kirill Stoimenov via Phabricator via llvm-commits]

kstoimenov added inline comments.


================
Comment at: llvm/lib/Target/X86/X86AsmPrinter.cpp:192-195
   case X86II::MO_GOTPCREL:  O << "@GOTPCREL";  break;
+  case X86II::MO_GOTPCREL_NORELAX: O << "@GOTPCREL_NORELAX"; break;
   case X86II::MO_GOT:       O << "@GOT";       break;
   case X86II::MO_GOTOFF:    O << "@GOTOFF";    break;
----------------
kstoimenov wrote:
> Maybe format the rest of the lines to align like they did before your change? Either that or format them to have a single space after the ";". 
Correction: after ":". 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113220