[all-commits] [llvm/llvm-project] 5d6d8d: [clang][llvm] Remove "implicit-section-name" attri...

[Arthur Eubanks via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5d6d8dcd292e0a107b11d378932eee9c2f9ccfc7
      https://github.com/llvm/llvm-project/commit/5d6d8dcd292e0a107b11d378932eee9c2f9ccfc7
  Author: Arthur Eubanks <aeubanks at google.com>
  Date:   2024-04-11 (Thu, 11 Apr 2024)

  Changed paths:
    M clang/lib/CodeGen/CodeGenModule.cpp
    M clang/test/CodeGen/clang-sections-attribute.c
    M clang/test/CodeGenCXX/clang-sections.cpp
    M llvm/lib/CodeGen/TargetInstrInfo.cpp
    M llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp
    M llvm/lib/IR/AutoUpgrade.cpp
    M llvm/lib/Target/TargetLoweringObjectFile.cpp
    A llvm/test/Bitcode/upgrade-implicit-section-name.ll
    R llvm/test/CodeGen/AArch64/clang-section-macho.ll
    M llvm/test/CodeGen/ARM/clang-section.ll
    M llvm/test/CodeGen/Generic/machine-function-splitter.ll
    M llvm/test/CodeGen/X86/basic-block-sections-pragma-sections.ll

  Log Message:
  -----------
  [clang][llvm] Remove "implicit-section-name" attribute (#87906)

D33412/D33413 introduced this to support a clang pragma to set section
names for a symbol depending on if it would be placed in
bss/data/rodata/text, which may not be known until the backend. However,
for text we know that only functions will go there, so just directly set
the section in clang instead of going through a completely separate
attribute.

Autoupgrade the "implicit-section-name" attribute to directly setting
the section on a Fuction.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications