AmrDeveloper wrote: > > This part in ObjectiveC tests conflict with the current solution, any > > suggestion how we can go around it right now until to be handled later > > https://github.com/llvm/llvm-project/blob/18952bdcd6f987620e6396261c2bb444e428e07e/clang/test/SemaObjCXX/noescape.mm#L202-L205 > > > > @erichkeane @AaronBallman > > Doesn't the fixme above that say it was diagnosing unintentionally? If so, > that fixme likely needs to be removed, and this is 'correct' now.
I handled it, i didn't know that `// expected-note n` is the number of expected notes, Thank you https://github.com/llvm/llvm-project/pull/111573 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
