[all-commits] [llvm/llvm-project] 45f883: Change check for embedded llvm version number to a...
dyung via All-commits
all-commits at lists.llvm.org
Fri Jan 26 09:36:32 PST 2024
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 45f883ed06f39fba7557dfbbff4d10595b45f874
https://github.com/llvm/llvm-project/commit/45f883ed06f39fba7557dfbbff4d10595b45f874
Author: dyung <douglas.yung at sony.com>
Date: 2024-01-26 (Fri, 26 Jan 2024)
Changed paths:
M llvm/test/CodeGen/SystemZ/zos-ppa2.ll
Log Message:
-----------
Change check for embedded llvm version number to a regex to make test more flexible. (#79528)
This test started to fail when LLVM created the release/18.x branch and
the main branch subsequently had the version number increased from 18 to
19.
I investigated this failure (it was blocking our internal automation)
and discovered that the CHECK statement on line 27 seemed to have the
compiler version number (1800) encoded in octal that it was checking
for. I don't know if this is something that explicitly needs to be
checked, so I am leaving it in, but it should be more flexible so the
test doesn't fail anytime the version number is changed. To accomplish
that, I changed the check for the 4-digit version number to be a regex.
I originally updated this test for the 18->19 transition in
a01195ff5cc3d7fd084743b1f47007645bb385f4. This change makes the CHECK
line more flexible so it doesn't need to be continually updated.
More information about the All-commits
mailing list