[PATCH] D69701: [Utils] Allow "on-the-fly" argument changes for update_test_check scripts

[Alexander Richardson via Phabricator via llvm-commits]

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

Looks good to me. Since this is in common.py, it would be nice to see it also applied to update_cc_test checks or update_llc_test_checks.



================
Comment at: llvm/test/tools/UpdateTestChecks/update_test_checks/basic.test:11
 # RUN: diff -u %t.ll %S/Inputs/basic.ll.funcsig.expected
-## Verify that running without the --function-signature flag removes the -SAME: lines:
+## Verify that running without the --function-signature flag does not removes the -SAME: lines:
 # RUN: %update_test_checks %t.ll
----------------
Maybe expand this comment to include "since the generated file will have --function-signature in an UTC_ARGS: comment" or similar


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69701