[all-commits] [llvm/llvm-project] d33abc: Migrate MCContext::createTempSymbol call sites to ...

[Fangrui Song via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d33abc337c74d03d4e49b8d81a2dba7f23594a1a
      https://github.com/llvm/llvm-project/commit/d33abc337c74d03d4e49b8d81a2dba7f23594a1a
  Author: Fangrui Song <i at maskray.me>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M llvm/lib/Target/ARM/ARMAsmPrinter.cpp
    M llvm/lib/Target/PowerPC/PPCPreEmitPeephole.cpp
    M llvm/test/CodeGen/PowerPC/p10-spill-crun.ll
    M llvm/test/CodeGen/PowerPC/pcrel-call-linkage-with-calls.ll
    M llvm/test/CodeGen/PowerPC/pcrel-got-indirect.ll
    M llvm/test/CodeGen/PowerPC/pcrel-linkeropt-option.ll
    M llvm/test/CodeGen/PowerPC/pcrel-linkeropt.ll
    M llvm/test/CodeGen/PowerPC/pcrel-relocation-plus-offset.ll
    M llvm/test/CodeGen/PowerPC/pcrel-tail-calls.ll
    M llvm/test/CodeGen/PowerPC/pcrel.ll

  Log Message:
  -----------
  Migrate MCContext::createTempSymbol call sites to AlwaysAddSuffix=true

Most call sites set AlwaysAddSuffix to true. The two use cases do not really
need false and can be more consistent with other temporary symbol usage.


  Commit: d9a0c40bce5f0b1325b89c36785d82fa146547aa
      https://github.com/llvm/llvm-project/commit/d9a0c40bce5f0b1325b89c36785d82fa146547aa
  Author: Fangrui Song <i at maskray.me>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M llvm/include/llvm/MC/MCContext.h
    M llvm/lib/CodeGen/MachineModuleInfo.cpp
    M llvm/lib/MC/MCContext.cpp
    M llvm/lib/MC/MCDwarf.cpp
    M llvm/lib/MC/MCObjectStreamer.cpp
    M llvm/lib/MC/MCSection.cpp
    M llvm/lib/MC/MCStreamer.cpp
    M llvm/lib/Target/ARM/ARMAsmPrinter.cpp
    M llvm/lib/Target/PowerPC/PPCPreEmitPeephole.cpp
    M llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp

  Log Message:
  -----------
  [MC] Split MCContext::createTempSymbol, default AlwaysAddSuffix to true, and add comments

CanBeUnnamed is rarely false. Splitting to a createNamedTempSymbol makes the
intention clearer and matches the direction of reverted r240130 (to drop the
unneeded parameters).

No behavior change.


Compare: https://github.com/llvm/llvm-project/compare/76f4f42ebaf9...d9a0c40bce5f