[all-commits] [llvm/llvm-project] b1932b: [MC] Aligned bundling: remove special handling for...

[Fangrui Song via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b1932b8483011c2bfebbea1ef56a565634570e6b
      https://github.com/llvm/llvm-project/commit/b1932b8483011c2bfebbea1ef56a565634570e6b
  Author: Fangrui Song <i at maskray.me>
  Date:   2024-06-14 (Fri, 14 Jun 2024)

  Changed paths:
    M llvm/include/llvm/MC/MCELFStreamer.h
    M llvm/include/llvm/MC/MCSection.h
    M llvm/include/llvm/MC/MCWasmStreamer.h
    M llvm/lib/MC/MCAssembler.cpp
    M llvm/lib/MC/MCELFStreamer.cpp
    M llvm/lib/MC/MCObjectStreamer.cpp
    M llvm/lib/MC/MCSection.cpp
    M llvm/lib/MC/MCWasmStreamer.cpp
    M llvm/test/MC/X86/AlignedBundling/misaligned-bundle-group.s
    M llvm/test/MC/X86/AlignedBundling/misaligned-bundle.s

  Log Message:
  -----------
  [MC] Aligned bundling: remove special handling for RelaxAll

When both aligned bundling and RelaxAll are enabled, bundle padding is
directly written into fragments (https://reviews.llvm.org/D8072).
(The original motivation was memory usage, which has been achieved from
different angles with recent assembler improvement).

The code presents challenges with the work to replace fragment
representation (e.g. #94950 #95077). This patch removes the special
handling. RelaxAll still works but the behavior seems slightly different
as revealed by 2 changed tests. However, most `-mc-relax-all` tests are
unchanged.

RelaxAll used to be the default for clang -O0. This mode has significant
code size drawbacks and newer Clang doesn't use it (#90013).

---

flushPendingLabels: The FOffset parameter can be removed: pending labels
will be assigned to the incoming fragment at offset 0.

Pull Request: https://github.com/llvm/llvm-project/pull/95188



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