[PATCH] D102243: [DAGCombiner] Add test exposing bug in DAG combine.
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 11 09:32:19 PDT 2021
spatel added a comment.
In D102243#2750964 <https://reviews.llvm.org/D102243#2750964>, @hgreving wrote:
> In D102243#2750702 <https://reviews.llvm.org/D102243#2750702>, @spatel wrote:
>
>> Is there some reason to not use scripted, auto-generated CHECK lines on this file?
>
> Would you mind hinting me to the script? I barely work upstream, sorry for that.
No problem - it's in the repo at:
utils/update_llc_test_checks.py ( https://github.com/llvm/llvm-project/blob/main/llvm/utils/update_llc_test_checks.py )
To run it, do something like this:
$ {path to}/update_llc_test_checks.py --llc={path to local}/llc dagcombine-dead-store.ll
Then, you can just run that same command after applying D102096 <https://reviews.llvm.org/D102096> and rebuilding llc. No need to manually touch the test file unless it looks wrong/excessive.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102243/new/
https://reviews.llvm.org/D102243
More information about the llvm-commits
mailing list