jamieschmeiser added a comment. This was originally approved and landed on May 11th. I agreed to let it be reverted when it was discovered that some headers were triggering the warning. I reworked the code to not generate the warning when coming from system header files and also added option control for the warnings. I was informed that the changes fixed the problems with the system headers but have received no other feedback since late May, despite numerous pings and requests tagging the various people involved. Since I have received no objections, further comments nor further review in approximately 2 months, I am assuming that there are no further concerns. Unless I hear otherwise, I will commit these changes tomorrow.
CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98798/new/ https://reviews.llvm.org/D98798 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
