<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Mar 19, 2018 at 9:59 AM Rafael Avila de Espindola via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">espindola added a comment.<br>
<br>
Some of these seem to be really unused (mutex in InputSection.cpp).<br>
<br>
Ideally we would remove just those.  Given that this changes just cpp files, I am somewhat tempted to LGTM this. If we want to avoid indirect includes, how dependable is IWYU these days?</blockquote><div><br>Not sure - I think some of the folks at Google are still really aiming to build something more reliable based on modules, etc.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Would the bots doing module builds find any problems?<br></blockquote><div><br>Some problems - though I think the modules builds only modularize the public (include/*) headers, not implementation headers.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D44509" rel="noreferrer" target="_blank">https://reviews.llvm.org/D44509</a><br>
<br>
<br>
<br>
</blockquote></div></div>