This is an archive of the discontinued LLVM Phabricator instance.

[Clang] Fix RHEL 6 build and other Include What You Use warnings
AbandonedPublic

Authored by Eugene.Zelenko on Jul 19 2016, 5:45 PM.

Details

Reviewers
hans
mehdi_amini
Summary

Build was broken because of missing climits where PATH_MAX is defined.

Diff Detail

Repository
rL LLVM

Event Timeline

Eugene.Zelenko retitled this revision from to [Clang] Fix RHEL 6 build and other Include What You Use warnings.
Eugene.Zelenko updated this object.
Eugene.Zelenko added reviewers: hans, mehdi_amini.
Eugene.Zelenko set the repository for this revision to rL LLVM.
Eugene.Zelenko added a subscriber: cfe-commits.
mehdi_amini added inline comments.Jul 19 2016, 6:29 PM
lib/Basic/FileManager.cpp
35

That's a lot of includes though. Ok for the climits, but the other should be a separate patch I think.

But all of them are used in code. I don't see a reason to save lines :-)

Eugene.Zelenko abandoned this revision.Aug 12 2016, 6:15 PM

Committed as fix for build failure after llvm/ADT/DenseMap.h in r278583.