[all-commits] [llvm/llvm-project] a1b496: (re-)Autogen one last unroll-and-jam test
Philip Reames via All-commits
all-commits at lists.llvm.org
Fri Nov 12 11:21:39 PST 2021
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: a1b496be6c7105943583acab0923acb8993e8e2f
https://github.com/llvm/llvm-project/commit/a1b496be6c7105943583acab0923acb8993e8e2f
Author: Philip Reames <listmail at philipreames.com>
Date: 2021-11-12 (Fri, 12 Nov 2021)
Changed paths:
M llvm/test/Transforms/LoopUnrollAndJam/unroll-and-jam.ll
Log Message:
-----------
(re-)Autogen one last unroll-and-jam test
This case was complicated because someone had added new non-autogened test to an autogened file. In particular, those new tests used two variables (%J and %j) which differeded only in capitalization. The auto-updater doesn't distinguish case, so this meant auto-gened versions of the new tests failed with non-obvious errors.
There are two key lessons here:
1) Please don't use two values which differ only in case. This is problematic for automatic tooling, but is also hard to understand for a human.
2) Please DO NOT add new tests to an autogened test without running autogen again. If autogen doesn't pass on your new test, put them in a separate file.
More information about the All-commits
mailing list