[all-commits] [llvm/llvm-project] 56cdd6: [MC] Merge fixupNeedsRelaxation and fixupNeedsRela...

[Fangrui Song via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 56cdd60a2f94d715c188ce17507796996d880f0d
      https://github.com/llvm/llvm-project/commit/56cdd60a2f94d715c188ce17507796996d880f0d
  Author: Fangrui Song <i at maskray.me>
  Date:   2026-03-05 (Thu, 05 Mar 2026)

  Changed paths:
    M llvm/include/llvm/MC/MCAsmBackend.h
    M llvm/lib/MC/MCAsmBackend.cpp
    M llvm/lib/Target/AArch64/MCTargetDesc/AArch64AsmBackend.cpp
    M llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUAsmBackend.cpp
    M llvm/lib/Target/CSKY/MCTargetDesc/CSKYAsmBackend.cpp
    M llvm/lib/Target/CSKY/MCTargetDesc/CSKYAsmBackend.h
    M llvm/lib/Target/M68k/MCTargetDesc/M68kAsmBackend.cpp

  Log Message:
  -----------
  [MC] Merge fixupNeedsRelaxation and fixupNeedsRelaxationAdvanced. NFC (#184832)

The two-tier relaxation hook design (fixupNeedsRelaxationAdvanced
delegating to fixupNeedsRelaxation) is confusing and unnecessary.

Only M68k and AMDGPU used fixupNeedsRelaxation (via the default
fixupNeedsRelaxationAdvanced). AArch64 and CSKY had dead overrides:
they don't override mayNeedRelaxation to ever return true.

Merge the two hooks: convert M68k and AMDGPU to override
fixupNeedsRelaxationAdvanced directly, remove dead overrides from
AArch64 and CSKY, and simplify the default implementation to
`return !Resolved`.



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