[PATCH] D83650: [FileCheck] Extend -dump-input with substitutions

[Joel E. Denny via Phabricator via llvm-commits]

jdenny added a comment.

Thanks for the review.



================
Comment at: llvm/test/FileCheck/dump-input-annotations.txt:635
+;
+; FIXME: Even CHECK-NOT should fail if a variable isn't defined, right?
+;--------------------------------------------------
----------------
thopre wrote:
> It should yes so you can rewrite the FIXME into something like "CHECK-NOT wrongly consider an undefined variable as a failure to match, fix first line input dump below once this is fixed".
> 
> IMHO, I think it should be fixed by making CheckNot handle UndefVarError errors and call a new printUndef and return true if any is found.
Let me know if the new text isn't what you had in mind.

There should probably be a bugzilla for this.  I'll try to look into it tomorrow.


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

https://reviews.llvm.org/D83650