[PATCH] D54769: [FileCheck] New option -warn

Joel E. Denny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 21 08:44:54 PST 2018


jdenny added a comment.

In https://reviews.llvm.org/D54769#1305396, @probinson wrote:

> This is small enough that I'd prefer to see it incorporated into the patch with the "real" warning, rather than have this temporary placeholder diagnostic.


Agreed, especially given that much of this patch will disappear at that point.  That might also help us to decide whether it's useful for PrintWarning to be exposed via the FileCheck class as it is now.


https://reviews.llvm.org/D54769





More information about the llvm-commits mailing list