<div dir="ltr">Would it be possible to configure this behavior? i.e. to have "fast but not silly isel", and "fast and debuggable isel"? The later should probably be the default, but some people (however silly they may be) want fast code from fast isel. ;-)</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 7, 2013 at 10:27 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">[+Eric, since I forgot him on the first mail]<br>
<br>
One other avenue that I haven't explored yet is that, perhaps,<br>
something like r182062 might be applicable, which simply retained more<br>
argument frame index information without generating more machine<br>
registers/instructions. I don't know at all if that applies to this<br>
issue, but it did address a similar issue of debug info missing on<br>
unused parameters (PR14575).<br>
<div><div class="h5"><br>
On Thu, Jun 6, 2013 at 7:52 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
> Hi Dan,<br>
><br>
> Here's the patch we talked about this afternoon - with the equivalent<br>
> change made to the ARM side of fast ISel to match the X86 change I'd<br>
> made. Turns out with the ARM change something does fail -<br>
> CodeGen/ARM/fast-isel-call.ll. So it seems this change isn't neutral<br>
> to codegen. (though I'm not sure how much we need to care deeply about<br>
> the performance of unused parameters?) I assume the source of the<br>
> change is the FIXME immediately proceeding the change to *FastISel.cpp<br>
> - so perhaps this is an acceptable "we'll fix the underlying issue &<br>
> this and some other things will get better".<br>
><br>
> I was wondering if you had any ideas on how this should be addressed<br>
> (is this acceptable? in which case I should just update the test case<br>
> (though I don't fully understand it yet). I've attached the two .s<br>
> files, with my change (a.s) and without my change (b.s) (around<br>
> r183465) along with the patch itself.<br>
><br>
> - David<br>
</div></div>_______________________________________________<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>
</blockquote></div><br></div>