My own comments:<br><br>This looks fine since it removes code duplication across multiple subprojects.  Have you tested that both libcxx and lldb work correctly when using this module as opposed to the existing ones?<br><div class="gmail_quote"><div dir="ltr">On Fri, Jun 10, 2016 at 2:15 AM Nitesh Jain <<a href="mailto:nitesh.jain@imgtec.com">nitesh.jain@imgtec.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">nitesh.jain added a comment.<br>
<br>
Hi Chandler Carruth,<br>
<br>
Please could you find some time to review this ?<br>
<br>
Thanks<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D20896" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20896</a><br>
<br>
<br>
<br>
</blockquote></div>