Re: [PATCH][ARM] Fix wrong code by arm_final_prescan with fp16 move instructions

2017-12-01 Thread Sudakshina Das
On 30/11/17 16:07, Kyrill Tkachov wrote: On 30/11/17 16:06, Sudakshina Das wrote: Hi Kyrill On 27/11/17 12:25, Kyrill Tkachov wrote: > Hi Sudi, > > On 24/11/17 14:57, Sudi Das wrote: >> Hi >> >> For the following test case: >> __fp16 >> test_select (__fp16 a, __fp16 b, __fp16 c) >> { >>ret

Re: [PATCH][ARM] Fix wrong code by arm_final_prescan with fp16 move instructions

2017-11-30 Thread Kyrill Tkachov
On 30/11/17 16:06, Sudakshina Das wrote: Hi Kyrill On 27/11/17 12:25, Kyrill Tkachov wrote: > Hi Sudi, > > On 24/11/17 14:57, Sudi Das wrote: >> Hi >> >> For the following test case: >> __fp16 >> test_select (__fp16 a, __fp16 b, __fp16 c) >> { >>return (a < b) ? b : c; >> } >> >> when compi

Re: [PATCH][ARM] Fix wrong code by arm_final_prescan with fp16 move instructions

2017-11-30 Thread Sudakshina Das
Hi Kyrill On 27/11/17 12:25, Kyrill Tkachov wrote: Hi Sudi, On 24/11/17 14:57, Sudi Das wrote: Hi For the following test case: __fp16 test_select (__fp16 a, __fp16 b, __fp16 c) { return (a < b) ? b : c; } when compiled with -mfpu=fp-armv8 -march=armv8.2-a+fp16 -marm -mfloat-abi=hard trunk

Re: [PATCH][ARM] Fix wrong code by arm_final_prescan with fp16 move instructions

2017-11-27 Thread Kyrill Tkachov
Hi Sudi, On 24/11/17 14:57, Sudi Das wrote: Hi For the following test case: __fp16 test_select (__fp16 a, __fp16 b, __fp16 c) { return (a < b) ? b : c; } when compiled with -mfpu=fp-armv8 -march=armv8.2-a+fp16 -marm -mfloat-abi=hard trunk generates wrong code: test_select: @ args

[PATCH][ARM] Fix wrong code by arm_final_prescan with fp16 move instructions

2017-11-24 Thread Sudakshina Das
Hi For the following test case: __fp16 test_select (__fp16 a, __fp16 b, __fp16 c) { return (a < b) ? b : c; } when compiled with -mfpu=fp-armv8 -march=armv8.2-a+fp16 -marm -mfloat-abi=hard trunk generates wrong code: test_select: @ args = 0, pretend = 0, frame = 0 @ frame_ne