[PATCH] D40850: suppress undefined-template warnings when the pattern is declared in a system header

2018-01-02 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Thanks, I agree with your reasoning. https://reviews.llvm.org/D40850 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D40850: suppress undefined-template warnings when the pattern is declared in a system header

2018-01-02 Thread Nick Lewycky via Phabricator via cfe-commits
nlewycky added a comment. Ping! https://reviews.llvm.org/D40850 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D40850: suppress undefined-template warnings when the pattern is declared in a system header

2017-12-05 Thread Nick Lewycky via Phabricator via cfe-commits
nlewycky created this revision. The way to fix an undefined-template warning is to add lines to the header file that defines the template pattern. We should suppress the warnings when the template pattern is in a system header because we don't expect users to edit those.