[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