djasper closed this revision.
djasper added a comment.
Submitted as r326023.
https://reviews.llvm.org/D43673
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Makes sense -- as dependencies can only be declared on top-level modules,
that's what we should be talking about here.
https://reviews.llvm.org/D43673
_
djasper created this revision.
djasper added a reviewer: rsmith.
All use declarations need to be directly placed in the top-level module anyway,
knowing the submodule doesn't really help. The header that has the offending
#include can easily be seen in the diagnostics source location.
https://