[PATCH] D44509: Remove unnecessary includes from lld/ELF

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 19 09:38:17 PDT 2018


dblaikie added a comment.

Generally, relying on indirect inclusion isn't ideal - if the  intermediate headers are later refactored & remove the includes, it may break the build of those depending on that indirect inclusion.

Was there a particular motivation for these changes?


Repository:
  rL LLVM

https://reviews.llvm.org/D44509





More information about the llvm-commits mailing list