https://github.com/haoNoQ commented:

The attribute gets some action! Nice!!

I noticed that currently the attribute's documentation doesn't say it can be 
placed on lambdas. But it doesn't look like it's actively rejected either. So 
it might be a good idea to update the documentation as part of the patch. 
Possibly even have a bit of a broader discussion about this - though I don't 
really see any downsides to simply allowing this.

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

Reply via email to