[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