[llvm] update_analyze_test_checks: support output from LAA (PR #67584)
Ramkumar Ramachandra via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 3 05:09:28 PDT 2023
artagnon wrote:
> Can you update all the test in, for example, llvm/test/Transforms/Attributor and check if there is churn.
> clang/test/OpenMP is another good place to check that.
There's a very small diff produced when I update all tests in llvm/test/Transforms/Attributor, and it's identical to the diff produced on main. clang/test/OpenMP is trickier, as very few of the tests have been auto-generated by update_cc_test_checks.py: nevertheless, I ran it on a few files by hand, and verified that the diff produced is identical to the one produced on main.
Thanks for suggesting the exercise: I'm now fairly confident that my patch doesn't generate churn.
https://github.com/llvm/llvm-project/pull/67584
More information about the llvm-commits
mailing list