[PATCH] D27655: Fix modernize-deprecated-headers clang-tidy warnings
Michael Sharpe via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 12 22:05:15 PST 2016
michael.sharpe added a comment.
In https://reviews.llvm.org/D27655#619307, @Eugene.Zelenko wrote:
> Did you enable analysis of headers in Clang-tidy?
I had not, thanks for the tip.
Comment at: lib/Lex/ModuleMap.cpp:35
> Could limits be used instead?
I built without this line on Ubuntu 16.04 without issue and I couldn't see where any limits were referenced from searching for _MAX or _MIN.
I think there might be a benefit to running include-what-you-use on this file, but I don't have the tool set up and will be without access to my personal setup for a couple weeks at the end of the year.
More information about the cfe-commits