[all-commits] [llvm/llvm-project] aa9cc7: Reapply "[BOLT] Add --pad-funcs-before=func:n (#11...

[Peter Waller via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: aa9cc721e58f086ba6a3f9711fefdb61a184f786
      https://github.com/llvm/llvm-project/commit/aa9cc721e58f086ba6a3f9711fefdb61a184f786
  Author: Peter Waller <peter.waller at arm.com>
  Date:   2025-01-07 (Tue, 07 Jan 2025)

  Changed paths:
    M bolt/lib/Core/BinaryEmitter.cpp
    M bolt/lib/Passes/ReorderFunctions.cpp
    A bolt/test/AArch64/pad-before-funcs.s

  Log Message:
  -----------
  Reapply "[BOLT] Add --pad-funcs-before=func:n (#117924)" (#121918)

- **Reapply "[BOLT] Add --pad-funcs-before=func:n (#117924)"**
- **[BOLT] Fix --pad-funcs{,-before} state misinteraction**

When --pad-funcs-before was introduced, it introduced a bug whereby the
first one to get parsed could influence the other.

Ensure that each has its own state and test that they don't interact in
this manner by testing how the `_subsequent` symbol moves when both
arguments are supplied with different padding values.

Fixed by having a function (and static state) for each of before/after.



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