<div dir="ltr"><div><div style="font-family:arial,sans-serif;font-size:13px">Hello Ahmed,</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 5, 2013 at 1:43 AM, Ahmed Bougacha <span dir="ltr"><<a href="mailto:ahmed.bougacha@gmail.com" target="_blank">ahmed.bougacha@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">On Thu, Jul 4, 2013 at 12:48 AM, Benoit Perrot <span dir="ltr"><<a href="mailto:benoit.noe.perrot@gmail.com" target="_blank">benoit.noe.perrot@gmail.com</a>></span> wrote:<br>
<div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><span style="color:rgb(80,0,80)"> </span><br>
</div></blockquote><div><div class="h5"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>


I have just updated my svn copy of the llvm/clang repositories after quite a long time of inactivity, and found it not compiling on Windows with Visual Studio 2008.</div>
<div>[...]</div><div>Solutions here:<br></div>


<div><br></div><div>1. consider that the implementation if effectively wrong, and modify the documentation at <a href="http://llvm.org/docs/GettingStartedVS.html" target="_blank">http://llvm.org/docs/GettingStartedVS.html</a>, requiring Visual Studio 2010, i.e. replacing:</div>



<div>2. investigate whether there exists a way to disable the aforementioned check;<br></div><div>3. modify the code in MCModule.cpp to cope with the implementation of "lower_bound" in VS 2008.<br></div>


<div><br></div><div>Personally I just went for (1), i.e. switching to Visual Studio 2010, as it was the most straightforward.</div></div></blockquote><div><br></div></div></div><div>(3) Fixed in r185676.</div><div><br></div>
<div>Requiring VS 2010 for a minor problem like this (even though there are more like it) isn’t warranted I think.</div></div></div></div></blockquote><div><br></div><div style>Great!</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="im">

<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>Doing so, I also had to add "#include <string>" to the file "lib/CodeGen/CGBlocks.cpp" so that llvm/clang can compile with said compiler, because of some obscure external template usage.</div>


</div></blockquote><div><br></div></div><div><string> is already included, at least by StringRef.h, so I’m curious: what is this obscure thing that needs including it again?</div></div><br></div></div>
</blockquote></div><br></div><div class="gmail_extra" style>My build was in an intermediate invalid state.</div><div class="gmail_extra" style>Reverting said modification, cleaning completely then building again just worked fine. Sorry for the false alarm.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>Thanks!</div><div class="gmail_extra" style>-- <br></div><div class="gmail_extra" style>Benoit PERROT</div></div>