[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