Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Example where bugpoint does a better job than llvm-reduce #54882

Closed
arsenm opened this issue Apr 13, 2022 · 2 comments
Closed

Example where bugpoint does a better job than llvm-reduce #54882

arsenm opened this issue Apr 13, 2022 · 2 comments
Labels
incomplete Issue not complete (e.g. missing a reproducer, build arguments, etc.) llvm-reduce llvm-tools All llvm tools that do not have corresponding tag

Comments

@arsenm
Copy link
Contributor

arsenm commented Apr 13, 2022

sample.zip
Here is a sample reduced testcase where bugpoint was able to improve upon llvm-reduce's result. After feeding llvm-reduce's output into bugpoint, it cut out a few hundred more lines of IR. It may or may not be useful to look at the diff to figure out what bugpoint is doing better.

The exact reproduction steps are hard to share. The original reproducer is a register allocation issue, which are always super sensitive to the exact code. So far I've only been able to reproduce it using a specific clang build (the one provided by the rocm 5.0 release version)

@EugeneZelenko EugeneZelenko added llvm-tools All llvm tools that do not have corresponding tag and removed new issue labels Apr 13, 2022
@regehr
Copy link
Contributor

regehr commented Jun 21, 2022

I don't think I'm going to try to reproduce this, but I'm very interested in specific things to add to llvm-reduce to help it get unstuck.

but also I just pushed a change to llvm-reduce that makes it run to fixpoint instead of running through its set of passes just once. unless you were previously using the -max-pass-iterations command line option, this new change might improve its output. if you don't mind building a fresh llvm and running llvm-reduce on your example again, I'd be interested to hear about the results

@arsenm
Copy link
Contributor Author

arsenm commented Dec 10, 2022

I don't see anything actionable here without a reproducer script. I do think llvm-reduce can cleanup some of the remaining things in this sample that it couldn't before.

@arsenm arsenm closed this as not planned Won't fix, can't repro, duplicate, stale Dec 10, 2022
@EugeneZelenko EugeneZelenko added the incomplete Issue not complete (e.g. missing a reproducer, build arguments, etc.) label Dec 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incomplete Issue not complete (e.g. missing a reproducer, build arguments, etc.) llvm-reduce llvm-tools All llvm tools that do not have corresponding tag
Projects
None yet
Development

No branches or pull requests

3 participants