<div dir="ltr"><div><div>Looks ok,  please commit it.<br><br></div>Thanks<br><br></div>Galina<br><br><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 25, 2016 at 5:19 AM, Vassil Vassilev <span dir="ltr"><<a href="mailto:v.g.vassilev@gmail.com" target="_blank">v.g.vassilev@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Galina,<br>
  I am attaching a patch that I believe fixes the issue. Could you review it please (I don't know how to test it).<br>
Many thanks,<br>
Vassil<div class="HOEnZb"><div class="h5"><br>
On 25/05/16 11:36, Vassil Vassilev wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This broke our modules slaves. I am working on a fix. Sorry for the inconvenience!<br>
-- Vassil<br>
On 24/05/16 11:23, Vassil Vassilev via llvm-commits wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: vvassilev<br>
Date: Tue May 24 04:22:58 2016<br>
New Revision: 270539<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=270539&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=270539&view=rev</a><br>
Log:<br>
Enable modules selfhosting builds with libstdc++, too.<br>
<br>
If a modulemap is not present (in the case of libstdc++), we will end up with<br>
duplicate entities in the modules. This will help us stress test the module<br>
merging and discover potential bugs.<br>
<br>
Modified:<br>
     zorg/trunk/zorg/buildbot/builders/ClangBuilder.py<br>
<br>
Modified: zorg/trunk/zorg/buildbot/builders/ClangBuilder.py<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/ClangBuilder.py?rev=270539&r1=270538&r2=270539&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/ClangBuilder.py?rev=270539&r1=270538&r2=270539&view=diff</a><br>
============================================================================== <br>
--- zorg/trunk/zorg/buildbot/builders/ClangBuilder.py (original)<br>
+++ zorg/trunk/zorg/buildbot/builders/ClangBuilder.py Tue May 24 04:22:58 2016<br>
@@ -281,8 +281,6 @@ def getClangBuildFactory(<br>
      cxx_flags = ''<br>
      extra_args = []<br>
      if modules:<br>
-        # Modules requires libc++ for now (we don't have a module map for libstdc++ yet).<br>
-        cxx_flags += '-stdlib=libc++'<br>
          extra_args = ['-DLLVM_ENABLE_MODULES=1']<br>
        f.addStep(ShellCommand(name='cmake',<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote>
<br>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div></div>