I ran into an error while trying to build libc++abi for a platform that
doesn't have <sys/types.h>. I couldn't find what <sys/types.h> was used
for in the header, so I think it's fine to remove it.
Details
Details
- Reviewers
EricWF compnerd - Group Reviewers
Restricted Project - Commits
- rGc2547f1554d3: [libc++abi] Remove unused include of <sys/types.h>
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
I agree, as long as declarations are available this should be fine. Even better - this is in the src tree, so we don't even have to worry about users.
Might be useful to just run IWYU over the tree at some point to remove extraneous headers.
Comment Actions
As long as no one deletes includes that may not be used, but are mandated by the standard.