https://github.com/AaronBallman commented:

Thank you for the fix!

We generally want ignored attributes to be diagnosed as being ignored; 
otherwise users have a much harder time determining whether the attribute is 
working or not. I think we should issue an "attribute ignored" warning when 
we're dropping the attribute in the AST.

Also, the changes should come with a release note so that users know about the 
bug fix.

https://github.com/llvm/llvm-project/pull/68059
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to