[all-commits] [llvm/llvm-project] 1fe463: [hexagon] Prevent alignment search beyond a label ...

[Alexey Karyakin via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1fe463182cead6e7c6119ab410eae9e9d969325a
      https://github.com/llvm/llvm-project/commit/1fe463182cead6e7c6119ab410eae9e9d969325a
  Author: Alexey Karyakin <akaryaki at quicinc.com>
  Date:   2025-03-11 (Tue, 11 Mar 2025)

  Changed paths:
    M llvm/lib/Target/Hexagon/MCTargetDesc/HexagonAsmBackend.cpp
    A llvm/test/MC/Hexagon/align-leb128.s
    M llvm/test/MC/Hexagon/align.s

  Log Message:
  -----------
  [hexagon] Prevent alignment search beyond a label (#130631)

When searching for packets to .align, don't consider ones which would
require padding beyond a label.

There are two problems with padding beyond a label:
- the distance between labels may increase for some offsets to become
too large;
- u/sleb128 values that encode a difference will not be updated because
they are computed before the align command is handled.

This is more a short-term fix/hack. The proper solution would be to
unify `.align` and `.falign` handling and move it to the layout loop.



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