jdoerfert wrote:

> > I think if the issues with the original commit are resolved, this is good 
> > to go.
> > Did you verify we can properly auto-generate files, e.g., in 
> > llvm/test/Transforms/Attributor and clang/test/OpenMP?
> 
> Ah no I did not, I'll do that on Monday.

I'd run `./llvm/utils/update_any_test_checks.py` once and see if the tests pass 
afterwards.
Then do it again to ensure the nasty ordering and duplication issues are gone 
for good.


https://github.com/llvm/llvm-project/pull/71171
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to