<div dir="ltr">lgtm<div><br></div><div>Yep, fastisel can definitely bail out. It happens a lot on Windows for unlowerable constructs.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 3, 2014 at 9:37 AM, Robinson, Paul <span dir="ltr"><<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.sony.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ping.<br>
<div class="HOEnZb"><div class="h5"><br>
> -----Original Message-----<br>
> From: Robinson, Paul<br>
> Sent: Sunday, October 26, 2014 9:23 PM<br>
> To: Robinson, Paul; <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> Subject: RE: [PATCH] Fix optnone on Windows<br>
><br>
> Ping.<br>
><br>
> > -----Original Message-----<br>
> > From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-">llvm-commits-</a><br>
> > <a href="mailto:bounces@cs.uiuc.edu">bounces@cs.uiuc.edu</a>] On Behalf Of Robinson, Paul<br>
> > Sent: Monday, October 20, 2014 3:55 PM<br>
> > To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> > Subject: [PATCH] Fix optnone on Windows<br>
> ><br>
> > [PATCH] Fix optnone on Windows<br>
> ><br>
> > Normally an 'optnone' function goes through fast-isel, which does not<br>
> > call DAGCombiner.  But we ran into a case (on Windows) where the<br>
> > calling convention causes argument lowering to bail out of fast-isel,<br>
> > and we end up in CodeGenAndEmitDAG() which does run DAGCombiner.<br>
> > So, we need to make DAGCombiner check for 'optnone' after all.<br>
> ><br>
> > Along the way we discovered we hadn't committed all the tests we had<br>
> > written for 'optnone' so here are two more. 'fastmath-optnone.ll' is<br>
> > the test that caught this, 'optnone-simple.ll' is one we missed from<br>
> > our original implementation.<br>
> > Thanks,<br>
> > --paulr<br>
<br>
</div></div><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>