AaronBallman wrote:

> IN SUMMARY: I think this proposal minus the std check is acceptable.

+1

> I think the proposed-not-implemented warning of "you messed with an attribute 
> name, and it is doing weird stuff" is also a very valuable, though different, 
> check.

I don't see them as different checks when they warn about the same thing, but I 
agree they can be separate implementations. I don't have a super strong feeling 
though because I've not really seen this problem crop up in the wild before 
except in contrived cases.

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

Reply via email to