[clang] [clang] assume_aligned incorrectly diagnoses a dependent return type (PR #111573)

Amr Hesham via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 9 09:48:19 PDT 2024


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


More information about the cfe-commits mailing list