This is an archive of the discontinued LLVM Phabricator instance.

Fix windows build after r285702, missing include to define PATH_MAX
ClosedPublic

Authored by Ilod on Nov 1 2016, 12:38 PM.

Details

Summary

Fix the windows build after r285702.
The removal of TimeValue.h inclusion removed the inner inclusion of PosixApi.h, which defined PATH_MAX on windows, used by many other files.
Adding the PosixApi.h inclusion should not cause a problem (it only include llvm/Support/Compiler.h on non-windows platform, and the needed file on windows ones).

Diff Detail

Event Timeline

Ilod updated this revision to Diff 76618.Nov 1 2016, 12:38 PM
Ilod retitled this revision from to Fix windows build after r285702, missing include to define PATH_MAX.
Ilod updated this object.
Ilod added reviewers: labath, zturner, clayborg.
Ilod added a subscriber: lldb-commits.
clayborg accepted this revision.Nov 1 2016, 3:24 PM
clayborg edited edge metadata.
This revision is now accepted and ready to land.Nov 1 2016, 3:24 PM
labath closed this revision.Nov 2 2016, 2:16 AM
labath edited edge metadata.

I believe Zachary already fixed this.