<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, May 18, 2018 at 12:46 PM Bruno Cardoso Lopes <<a href="mailto:bruno.cardoso@gmail.com">bruno.cardoso@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, May 18, 2018 at 11:54 AM Vedant Kumar via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><blockquote type="cite">On May 18, 2018, at 11:48 AM, Eric Liu <<a href="mailto:ioeric@google.com" target="_blank">ioeric@google.com</a>> wrote:<br></blockquote><div><blockquote type="cite"><br class="gmail-m_-3115138368819719797Apple-interchange-newline"><div><div dir="ltr">So I have reverted this with r<span style="color:rgb(33,33,33)">332751</span><span style="color:rgb(33,33,33)">.</span></div></div></blockquote><div><br></div>Thanks!</div><div><br></div><div><br><blockquote type="cite"><div><div dir="ltr"><div><font color="#212121">I can't see how this introduced cyclic dependencies in module build, as the dependencies should be clangTooling -> clangFormat -> clangToolingInclusions. I'm wondering if there is any module configurations that I need to update to make this work. Right now, module doesn't seem to make any difference between </font><span style="color:rgb(33,33,33)">clangTooling and </span><span style="color:rgb(33,33,33)">clangToolingInclusions... I'd appreciate it if someone who knows how clang module build is set up could help take a look.</span></div></div></div></blockquote><div><br></div><div>+ Bruno & David who have more experience in this area than I do.</div></div></div></blockquote><div><br></div><div>Gonna try to reproduce and take a look!</div></div></div></blockquote><div><br></div><div>I could reproduce it. You should be good to go if you add another top level module for Inclusions (and break the dep):</div><div><br></div><div><div>--- a/include/clang/module.modulemap</div><div>+++ b/include/clang/module.modulemap</div><div>@@ -153,3 +153,8 @@ module Clang_ToolingCore {</div><div>   requires cplusplus</div><div>   umbrella "Tooling/Core" module * { export * }</div><div> }</div><div>+</div><div>+module Clang_ToolingInclusions {</div><div>+  requires cplusplus</div><div>+  umbrella "Tooling/Inclusions" module * { export * }</div><div>+}</div></div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature">Bruno Cardoso Lopes <br><a href="http://www.brunocardoso.cc">http://www.brunocardoso.cc</a></div></div>