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

Alexey Karyakin via All-commits all-commits at lists.llvm.org
Tue Mar 25 16:15:03 PDT 2025


  Branch: refs/heads/release/20.x
  Home:   https://github.com/llvm/llvm-project
  Commit: e0e8071815c76b935c2145699c72bf833f0d0af5
      https://github.com/llvm/llvm-project/commit/e0e8071815c76b935c2145699c72bf833f0d0af5
  Author: Alexey Karyakin <akaryaki at quicinc.com>
  Date:   2025-03-25 (Tue, 25 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.

(cherry picked from commit 1fe463182cead6e7c6119ab410eae9e9d969325a)



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


More information about the All-commits mailing list