[llvm-commits] [LLVM, SelectionDAG] fix for #9905: Failure in code selection for llvm intrinsics sqrt/exp

Eli Friedman eli.friedman at gmail.com
Wed Dec 7 17:42:35 PST 2011


On Wed, Dec 7, 2011 at 2:42 AM, Stepan Dyatkovskiy <stpworld at narod.ru> wrote:
> Please, look at fixed patch in attachment.

Looks fine.  Do you need me to commit this for you?

-Eli

> Thanks.
> -Stepan.
>
>
> Eli Friedman wrote:
>>
>> On Tue, Dec 6, 2011 at 1:46 AM, Stepan Dyatkovskiy<stpworld at narod.ru>
>>  wrote:
>>>
>>> Sorry for previous post. Forgot to remove extra newlines at the end of
>>> file.
>>> Fixed file is attached here.
>>
>>
>> Better... but please try to make it less sensitive to register
>> allocation and scheduler choices.
>>
>> -Eli
>>
>>>
>>> Thanks.
>>> -Stepan.
>>>
>>>
>>> Stepan Dyatkovskiy wrote:
>>>>
>>>>
>>>> OK. Please look at reworked regression test patch in attachment.
>>>>
>>>> -Stepan.
>>>>
>>>> Eli Friedman wrote:
>>>>>
>>>>>
>>>>> The code changes look fine. Please put all the FileCheck tests into
>>>>> one file, and only use CHECK lines for the most important pieces
>>>>> (specifically, that we call sinf etc.).
>>>>>
>>>>> -Eli
>>>>>
>>>>> On Mon, Dec 5, 2011 at 5:05 AM, Stepan Dyatkovskiy<stpworld at narod.ru>
>>>>> wrote:
>>>>>>
>>>>>>
>>>>>> ping.
>>>>>>
>>>>>> -Stepan.
>>>>>>
>>>>>> Stepan Dyatkovskiy wrote:
>>>>>>>
>>>>>>>
>>>>>>> ping.
>>>>>>>
>>>>>>> -Stepan
>>>>>>>
>>>>>>> Stepan Dyatkovskiy wrote:
>>>>>>>>
>>>>>>>>
>>>>>>>> Hi all. Please find the patch and regression tests in attachment for
>>>>>>>> review.
>>>>>>>> This patch for ARM. It fixes selection for several instructions that
>>>>>>>> works with v4f32 arguments: FSQRT, FSIN, FCOS, FPOWI, FPOW, FLOG,
>>>>>>>> FLOG2,
>>>>>>>> FLOG10, FEXP, FEXP2.
>>>>>>>> I'm still worrying about FCOPYSIGN, FNEG, FABS, FCEIL, FTRUNC,
>>>>>>>> FRINT,
>>>>>>>> FNEARBYINT, FFLOOR. I could not found a way to add this instructions
>>>>>>>> with v2f32 argument to DAG. It seems that it is impossible in ToT.
>>>>>>>> So
>>>>>>>> these instructions was not fixed.
>>>>>>>>
>>>>>>>> -Stepan.
>>>>>>>>
>>>>>>>>
>>>>>>>> _______________________________________________
>>>>>>>> llvm-commits mailing list
>>>>>>>> llvm-commits at cs.uiuc.edu
>>>>>>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> llvm-commits mailing list
>>>>>>> llvm-commits at cs.uiuc.edu
>>>>>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>>>>>
>>>>>>
>>>>>>
>>>>>> _______________________________________________
>>>>>> llvm-commits mailing list
>>>>>> llvm-commits at cs.uiuc.edu
>>>>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> llvm-commits mailing list
>>>> llvm-commits at cs.uiuc.edu
>>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>>
>>>
>>>
>




More information about the llvm-commits mailing list