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

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 14 15:17:55 PST 2021


jdoerfert reopened this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93078



More information about the cfe-commits mailing list