[PATCH] D118364: [BOLT] Remove peephole useless-branches pass

Amir Ayupov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 28 12:39:42 PST 2022


Amir added a comment.

In D118364#3277620 <https://reviews.llvm.org/D118364#3277620>, @Amir wrote:

> Hi Vladislav,
>
> I've added the clang-icp.test to binary tests repo (https://github.com/rafaelauler/bolt-tests) and this test breaks with this diff. Please let me know if you can reproduce it.

Please use bolt/utils/llvm-bolt-wrapper.py script (rGc2a961e414e0 <https://reviews.llvm.org/rGc2a961e414e07e3657f56156cbda8cd0e2e7f9b7>) to run binary matching.

1. Build BOLT without this diff, rename the binary as llvm-bolt.old
2. Build BOLT with this diff, rename the binary as llvm-bolt.new
3. Run the script to produce llvm-bolt-wrapper.ini:

`/path/to/llvm-bolt-wrapper.py /path/to/llvm-bolt.old /path/to/llvm-bolt.new > bin/llvm-bolt-wrapper.ini`

4. Symlink the script in place of llvm-bolt:

`ln -sf /path/to/llvm-bolt-wrapper.py /path/to/llvm-bolt`

5. Run binary test:

`bin/llvm-lit -a tools/bolttests/clang-icp.test`

The mismatch in text section should be reported.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118364/new/

https://reviews.llvm.org/D118364



More information about the llvm-commits mailing list