[PATCH] D93078: [utils] Fix UpdateTestChecks case where 2 runs differ for last label

Thu Jan 14 18:46:20 PST 2021

mtrofin added a comment.

In D93078#2499666 <https://reviews.llvm.org/D93078#2499666>, @jdoerfert wrote:

> I think this caused a lot of problems for the Attributor tests. I get the "conflict output" warning all the time now :(
>
> These two `UpdateTestChecks` tests also generate the warning. I would think that they haven't before.
>
>   LLVM :: tools/UpdateTestChecks/update_test_checks/check_attrs.test
>   LLVM :: tools/UpdateTestChecks/update_test_checks/prefix-never-matches.test
>
> I suspect the `--check-attributes` flag effect are not taken into account somewhere but I might be wrong.

The warnings are correct, but their purpose is to inform. For example, take check_attrs. The third and fourth opt run produce different function attributes from those produced by the first two, so the function is different insofar as update_test_prefix is concerned. The warnings are there to indicate that the listed prefixes end up not being used. But, that's 'by design' for the attributor tests, IIRC.

If the warnings are noise for you, we could add a flag to quiet them down, which you'd flip when regenerating attributor tests?

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93078/new/

https://reviews.llvm.org/D93078