[PATCH] D85415: [Sparc] Don't use SunStyleELFSectionSwitchSyntax

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 13 12:23:11 PDT 2020


efriedma added a comment.

> Instead of adapting the affected tests, this patch changes that default. The internal assembler still accepts both forms as input, only the output syntax is affected.

There's a check in the parser for usesSunStyleELFSectionSwitchSyntax(); what effect does that have?

If we expect that everyone targeting SPARC these days is using an assembler that understands the standard ELF syntax, printing that syntax seems fine.  I assume that's true on Linux; not sure what assemblers are commonly used on Solaris.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85415



More information about the llvm-commits mailing list