<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, May 7, 2014 at 9:12 AM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br>
<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 class="">David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> writes:<br>

> On Wed, May 7, 2014 at 2:00 AM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:<br>
>> I've worked around that in r208171.<br>
>><br>
>> Please take a look and come up with a more permanent fix.<br>
><br>
> This is essentially the semi-permanent fix (Justin can just remove the<br>
> comment). Reid's done this in other places for the same reason.<br>
><br>
> One discussion we had is that we could modify STLExtras.h to check<br>
> compiler version and just bring in the MSVC version of make_unique<br>
> into the llvm namespace if it's available, and otherwise define our<br>
> own. Then there wouldn't be an ambiguity.<br>
<br>
</div>I've removed the comment and fully qualified the two other calls to<br>
make_unique in the same function in r208219. Two questions:<br>
<br>
1. I guess we don't have buildbots with the right version of MSVC to hit<br>
   this?  I didn't get any email.<br></blockquote><div><br></div><div>We don't have any bots running MSVC 2013.  We support building LLVM with two released versions of MSVC today: 2012 and 2013.  This problem exists with 2013.</div>
<div> </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">
2. Why wasn't the zero argument call to make_unique ambiguous?<br></blockquote><div><br></div><div>¯\(°_o)/¯ </div></div><br></div></div>