<div dir="ltr">Agreed. That was probably just luck before :)<div><br></div><div>-eric</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, May 12, 2017 at 5:22 AM Diana Picus via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It turns out that can be fixed by adding -lm to the link line, so I<br>
will probably convert it into a test-suite bug.<br>
<br>
I don't suppose it's crucial to handle the fabs intrinsic nicely at -O0.<br>
<br>
On 12 May 2017 at 13:44, Diana Picus <<a href="mailto:diana.picus@linaro.org" target="_blank">diana.picus@linaro.org</a>> wrote:<br>
> Hi,<br>
><br>
> I ran into a little snag on the test-suite:<br>
> <a href="https://bugs.llvm.org/show_bug.cgi?id=33021" rel="noreferrer" target="_blank">https://bugs.llvm.org/show_bug.cgi?id=33021</a><br>
> It boils down to GlobalISel generating calls to fabs instead of using<br>
> FABSDr (so we get undefined references).<br>
><br>
> Cheers,<br>
> Diana<br>
><br>
> On 11 May 2017 at 18:40, Quentin Colombet <<a href="mailto:qcolombet@apple.com" target="_blank">qcolombet@apple.com</a>> wrote:<br>
>> Hi Diana,<br>
>><br>
>> Thanks for the summary.<br>
>><br>
>> On May 11, 2017, at 4:01 AM, Diana Picus <<a href="mailto:diana.picus@linaro.org" target="_blank">diana.picus@linaro.org</a>> wrote:<br>
>><br>
>> Hi all,<br>
>><br>
>> I'm still running some validation on this, I'll send an email when<br>
>> it's done. If that goes well I don't have anything against making the<br>
>> switch.<br>
>><br>
>> For the record, here's a summary of issues that were deferred for<br>
>> later on (some of which are optimization-ish and we might decide to<br>
>> never do at -O0 at all):<br>
>> * Crash in RegBankSelect for half fp types:<br>
>> <a href="https://bugs.llvm.org/show_bug.cgi?id=32560" rel="noreferrer" target="_blank">https://bugs.llvm.org/show_bug.cgi?id=32560</a><br>
>><br>
>><br>
>> I’ll have a look.<br>
>><br>
>> * Improving constant placement: <a href="http://bugs.llvm.org/show_bug.cgi?id=32561" rel="noreferrer" target="_blank">http://bugs.llvm.org/show_bug.cgi?id=32561</a><br>
>><br>
>><br>
>> I’ve commented in the PR to mention the localizer technic I was playing<br>
>> with, if someone wants to give it a try.<br>
>><br>
>> * Fancy switch lowering<br>
>> * Transforming division-by-constant-power-of-2 into right shift<br>
>><br>
>><br>
>> AFAICT all the other issues that were brought up were fixed (yay!).<br>
>><br>
>> Cheers,<br>
>> Diana<br>
>><br>
>><br>
>> Cheers,<br>
>> -Quentin<br>
>><br>
>><br>
>><br>
>> On 11 May 2017 at 08:44, Kristof Beyls via llvm-dev<br>
>> <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>><br>
>><br>
>> On 10 May 2017, at 17:36, Quentin Colombet <<a href="mailto:qcolombet@apple.com" target="_blank">qcolombet@apple.com</a>> wrote:<br>
>><br>
>><br>
>> MultiSource/Benchmarks/mediabench/mpeg2/mpeg2dec/mpeg2decode (46%): Function<br>
>> Reference_IDCT: Probably due to creating all constants in the entry BB +<br>
>> spilling floating point data through an X register:<br>
>><br>
>> FastISel:<br>
>> fadd d0, d1, d0<br>
>> str d0, [sp,#528]<br>
>> GlobalISel:<br>
>> fadd d0, d1, d0<br>
>> fmov x9, d0<br>
>> stur x9, [x29,#-48]<br>
>><br>
>><br>
>> Good finding, I forgot to do stores in my previous fix. I’ll do them<br>
>> shortly.<br>
>><br>
>><br>
>> Should be fixed by r302679<br>
>><br>
>><br>
>> Thanks Quentin,<br>
>><br>
>> That reduces the slow-down when enabling globalisel at -O0 from 13% (on<br>
>> r302453)  to 9.5% (on r302679) in my experiments.<br>
>> The code size increase also reduces from just over 3% to 2.8%.<br>
>><br>
>> Kristof<br>
>><br>
>><br>
>> _______________________________________________<br>
>> LLVM Developers mailing list<br>
>> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
>><br>
>><br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>