Clang compiles and test passes without these includes.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
- Build Status
Buildable 16152 Build 16152: arc lint + arc unit
Event Timeline
Comment Actions
Shouldn't the Translation Unit be responsible for knowing what it needs rather than hoping it's already been used somewhere up the chain.
Perhaps std::unique_ptr and std::make_pair are too low level and one can assume, but D44509 seems like a possibly overzealous application?
Comment Actions
@marsupial Thanks for the commment, found a bug in my script and fixed. Now this patch removes only includes which are not used in each TU.