lebedev.ri added a comment.

In https://reviews.llvm.org/D36892#856501, @JonasToth wrote:

> Note can be handled right now as well.


Yes. Adding this new prefix is about adding `-implicit-check-not=notes`.
I.e. if you use `CHECK-MESSAGES`, then it will only enforce you to have 
check-lines for all the `error: ` and `warning: ` the check outputs.
It does *not* enforce that all the `note: ` are to be handled. `CHECK-NOTES` 
however would enforce that.

> E.g.
> 
>   // CHECK-MESSAGES: [[@LINE-2]]:3: note: type deduction did not result in an 
> owner
> 
> would the patch handle the codelocation correctly?

I'm not sure what is the question to be honest. This does not change anything 
about codelocation handling


Repository:
  rL LLVM

https://reviews.llvm.org/D36892



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to