[all-commits] [llvm/llvm-project] d99934: [Sparc] Don't use SunStyleELFSectionSwitchSyntax

rorth via All-commits all-commits at lists.llvm.org
Wed Aug 17 04:00:10 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d9993484ee40abaab6d9e96872d3a29044202aea
      https://github.com/llvm/llvm-project/commit/d9993484ee40abaab6d9e96872d3a29044202aea
  Author: Rainer Orth <ro at gcc.gnu.org>
  Date:   2022-08-17 (Wed, 17 Aug 2022)

  Changed paths:
    M llvm/lib/MC/MCParser/ELFAsmParser.cpp
    M llvm/lib/Target/Sparc/MCTargetDesc/SparcMCAsmInfo.cpp
    M llvm/test/CodeGen/SPARC/2008-10-10-InlineAsmRegOperand.ll
    M llvm/test/CodeGen/SPARC/constructor.ll

  Log Message:
  -----------
  [Sparc] Don't use SunStyleELFSectionSwitchSyntax

As discussed in D85414 <https://reviews.llvm.org/D85414>, two tests
currently `FAIL` on Sparc since that backend uses the Sun assembler syntax
for the `.section` directive, controlled by
`SunStyleELFSectionSwitchSyntax`.

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.

Current support for the Sun syntax is cursory at best: the built-in
assembler cannot even assemble some of the directives emitted by GCC, and
the set supported by the Solaris assembler is even larger: SPARC Assembly
Language Reference Manual, 3.4 Pseudo-Op Attributes
<https://docs.oracle.com/cd/E37838_01/html/E61063/gmabi.html#scrolltoc>.

A few Sparc test cases need to be adjusted. At the same time, the patch
fixes the failures from D85414 <https://reviews.llvm.org/D85414>.

Tested on `sparcv9-sun-solaris2.11`.

Differential Revision: https://reviews.llvm.org/D85415




More information about the All-commits mailing list