<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Aug 16, 2017 at 2:30 PM, Reid Kleckner via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</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"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Wed, Aug 16, 2017 at 11:21 AM, Stephan T. Lavavej <span dir="ltr"><<a href="mailto:stl@exchange.microsoft.com" target="_blank">stl@exchange.microsoft.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">[STL]<br>
<span>> Here's how to repro my directory structure in a clean VM.<br>
<br>
</span>[don hinton]<br>
<span>> Currently, for pre-2017, the root directory must be in the form <some path>/VC.  <br>
<br>
</span>Note that these development builds (both my actual ones, and the repro constructed from the nuget package) are 2017-class. They just don't conform to 2017's normal installer.<br>
<span><br>
> Could you try renaming x86ret and amd64ret to VC and try it again? e.g.:<br>
> C:\Temp\binaries\x86ret\bin\i3<wbr>86       ==> C:\Temp\binaries\VC\bin\i3<wbr>86<br>
> C:\Temp\binaries\amd64ret\bin\<wbr>amd64    ==> C:\Temp\binaries\VC\bin\a<wbr>md64<br>
<br>
</span>Sure. x86 still misbehaves, but x64 works properly! Does this indicate that Clang just needs to do slightly less strict validation of the root directory (possibly controlled by an option)?<br></blockquote><div><br></div></span><div>I'm in favor of this. If we find a cl.exe on PATH, we can take its version and use that to set our compatibility mode flags. We don't need to validate that it looks like its in a VC installation tree, and I'd rather not have an option for it. Whatever we use the knowledge of the VC tree root for would probably not work though. We won't be able to find and prefer the x64/x86 cross-linker, for example.</div></div></div></div></blockquote><div><br></div><div>Isn't "cl.exe on path" the common case for people running vcvarsall? Wouldn't we want to validate in that case? (If not, why do it ever?)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>______________________________<wbr>_________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div></div>