<div dir="rtl"><div dir="ltr">Hi,</div><div dir="ltr"><br></div><div dir="ltr">Rafael reverted the patch.  I have now fixed both tests:</div><div dir="ltr"><br></div><div dir="ltr">1) Added back the !Subtarget->isTargetEnvMacho() condition, this fixed CodeGen\X86\win64_alloca_dynalloca.ll.</div>

<div dir="ltr"><br></div><div dir="ltr">2) modified the second patch from isTargetWindows() to isOSWindows(). These two are not the same - how confusing!!, this fixed CodeGen\X86\mingw-alloca.ll.<br><br>I'll commit soon.<br>

<br><div dir="ltr">Yaron<br></div></div><div dir="ltr"><br></div><div dir="ltr"><br></div><div dir="ltr"><br></div><div class="gmail_extra" dir="ltr"><div class="gmail_quote"><div>2013/10/24 Anton Korobeynikov <span dir="ltr"><<a href="mailto:anton@korobeynikov.info" target="_blank">anton@korobeynikov.info</a>></span></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 class="im">> It's easy to bring back the  !Subtarget->isTargetEnvMacho()  condition but<br>


> is it correct?<br>
</div>For now - yes. Looks like macho environment was used to build some EFI code...<br>
<div class="im"><br>
> mingw32 is COFF by default (unless -elf) so results should have been the<br>
> same as before, right?<br>
</div>Yes.<br>
<div class="im"><br>
> Do the standard regression tests run at all under Windows? I'll try building<br>
> them right now.<br>
><br>
> I'm fairly new at this, sorry for any trouble!<br>
</div>No problem. You may want to revert your change for a moment, fix the<br>
problem and then recommit.<br>
<div class=""><div class="h5"><br>
--<br>
With best regards, Anton Korobeynikov<br>
Faculty of Mathematics and Mechanics, Saint Petersburg State University<br>
</div></div></blockquote></div><br></div></div>