[PATCH] D142473: [UTC] Add --version argument
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 24 08:44:31 PST 2023
asb added a comment.
In D142473#4077336 <https://reviews.llvm.org/D142473#4077336>, @nhaehnle wrote:
> - In the past, I have on occasion made commits that only re-run UTC on existing tests, without making any other changes, and without posting them for review.
> - I added `update_any_test_checks.py` in part to make that sort of change easier (though the primary motivation was to just update tests after code changes)
> - I wonder if / to what extent normalizing and perhaps automating this process is an alternative solution.
This is a good point, and worrying less about churn to existing tests is I think not a bad option. It's natural to want to compare such changes to other automated changes like code reformatting, but I think in practice it's very different - as the only changes are to tool-generated lines, it doesn't really cause the same issues with merge conflicts.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142473/new/
https://reviews.llvm.org/D142473
More information about the llvm-commits
mailing list