whisperity added a comment. In D91000#3770071 <https://reviews.llvm.org/D91000#3770071>, @whisperity wrote:
> In D91000#3197851 <https://reviews.llvm.org/D91000#3197851>, @aaron.ballman > wrote: > >> In terms of whether we should expose the check in C++: I think we shouldn't. >> [...] >> >> I think we should probably also not enable the check when the user compiles >> in C99 or earlier mode, because there is no Annex K available to provide >> replacement functions. > > @aaron.ballman I think the current version of the check satisfies these > conditions. It seems the check **will** run for every TU, but in case there > is no alternative the check could suggest, it will do nothing: > > if (!ReplacementFunctionName) > return; > > Is this good enough? This seems more future-proof than juggling the > `LangOpts` instance in yet another member function. @aaron.ballman @njames93 Ping! It seems @futogergely has resigned from the company, so I'll end up flying the approach, but the one above is the last outstanding question. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91000/new/ https://reviews.llvm.org/D91000 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
