[llvm] [MC] Parse SHF_LINK_ORDER argument before section group name (PR #77407)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 9 00:13:55 PST 2024


================
@@ -90,7 +90,9 @@ void MCSectionELF::printSwitchToSection(const MCAsmInfo &MAI, const Triple &T,
     OS << 'e';
   if (Flags & ELF::SHF_EXECINSTR)
     OS << 'x';
-  if (Flags & ELF::SHF_GROUP)
+  // TODO: Always print G after o to be clear that the 'G' argument is parsed
----------------
MaskRay wrote:

Thanks for the quick review. I have just created #77422 to clean up this and update some tests (not that many), but I want to keep them separate from this change.

https://github.com/llvm/llvm-project/pull/77407


More information about the llvm-commits mailing list