chandlerc added a comment.

I think we want dedicated macros to disable each added [[nodiscard]]. The 
reason is that as you add each one, users will have to roll out cleanups for 
the new warnings. This may be hard to do quickly and cause problems updating 
libc++ for users with -Werror in their build. I'd prefer it if users can 
initially add a macro to disable the attribute until their cleanup is complete.

There should also be a page documenting all of the APIs that use this and the 
macros to control them.


https://reviews.llvm.org/D26596



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to