Sorry, missed the patch.<div><br></div><div>LGTM, please commit.<br><br><div class="gmail_quote">On Tue, Jun 5, 2012 at 6:15 PM, Justin Holewinski <span dir="ltr"><<a href="mailto:justin.holewinski@gmail.com" target="_blank">justin.holewinski@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Anyone against committing this?<div class="HOEnZb"><div class="h5"><br><br><div class="gmail_quote">On Mon, Jun 4, 2012 at 11:02 PM, Justin Holewinski <span dir="ltr"><<a href="mailto:justin.holewinski@gmail.com" target="_blank">justin.holewinski@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">An apparent change to the Visual C++ STL for Visual Studio 2012 is causing gtest to no longer build.  gtest requires std::tr1::tuple to support up to 10 template parameters, but the implementation in Visual Studio 2012 RC only supports up to 8 by default.  This patch adds a compiler definition (-D_VARIADIC_MAX=10) to the gtest libs and unit tests when MSVC_VERSION == 1700.<div>


<br></div><div>Upstream issue:  
<a href="http://code.google.com/p/googletest/issues/detail?id=408" target="_blank">http://code.google.com/p/googletest/issues/detail?id=408</a> <span><font color="#888888"><br clear="all">
<div><br></div>-- <br><br><div>Thanks,</div><div><br></div><div>Justin Holewinski</div><br>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><br><div>Thanks,</div><div><br></div><div>Justin Holewinski</div><br>
</font></span><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>