Endill added a comment.
Thank you for your time!
================
Comment at: clang/lib/Frontend/VerifyDiagnosticConsumer.cpp:468
+ Status = VerifyDiagnosticConsumer::HasExpectedMaybeNoDiagnostics;
+ continue;
+ } else if (DToken.endswith(DType="-no-diagnostics")) {
----------------
jdenny wrote:
> Endill wrote:
> > jdenny wrote:
> > > Endill wrote:
> > > > jdenny wrote:
> > > > > This `continue` skips the prefix checking below, which is important
> > > > > when there are multiple prefixes active (e.g., `-verify=foo,bar`).
> > > > > That is, any old `BOGUS-maybe-no-diagnostics` will be effective then.
> > > > This should be fixed now. Thank you for spotting this!
> > > Thanks for the fix. Please add a test so this bug doesn't pop up again.
> > Done as A2 test
> Does A2 trigger the original bug? I think there must be multiple prefixes
> for this fix to matter.
I think prefix checking below works for a single prefix as well:
https://github.com/llvm/llvm-project/blob/5217498dc88aa2de2b728462205ffa8b01d96cab/clang/lib/Frontend/CompilerInvocation.cpp#LL2382C47-L2382C47
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151320/new/
https://reviews.llvm.org/D151320
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
