[PATCH] D80584: [utils] change update_test_checks.py use of 'TMP' value names

Sanjay Patel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 1 04:46:43 PDT 2020


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

Effectively reverted (but still have the name conflict warning at least) with:
rGe5b877275673 <https://reviews.llvm.org/rGe5b8772756737e41cb1e8ee1a5a33cb3d8a25be6>

If we can change instnamer, we can probably keep living with this script as-is. Alternatively, let's re-fix this script and mass update test files, so there's no general concern about churn of existing tests.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80584





More information about the cfe-commits mailing list