[PATCH] D68081: Allow update_test_checks.py to not scrub names

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 26 14:25:02 PDT 2019


reames added a comment.

In D68081#1684028 <https://reviews.llvm.org/D68081#1684028>, @lebedev.ri wrote:

> I'm not sure this is a good idea.


I also don't see the value in this.  With autogenerated checks, the name of the instructions shouldn't be required to tell if the test is getting the correct behaviour and preserving the names makes test checks much more fragile.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68081





More information about the llvm-commits mailing list