Re: [PATCH][simplify-rtx] Zero-initialise local array in simplify_immed_subreg

2016-10-10 Thread Kyrill Tkachov
On 07/10/16 18:56, Andrew Pinski wrote: On Fri, Oct 7, 2016 at 10:55 AM, Andrew Pinski wrote: On Fri, Oct 7, 2016 at 7:08 AM, Kyrill Tkachov wrote: Hi all, I've encountered another wrong-code bug with the store merging pass. This time it's in RTL. The test gcc.target/aarch64/aapcs64/test_27

Re: [PATCH][simplify-rtx] Zero-initialise local array in simplify_immed_subreg

2016-10-07 Thread Andrew Pinski
On Fri, Oct 7, 2016 at 10:55 AM, Andrew Pinski wrote: > On Fri, Oct 7, 2016 at 7:08 AM, Kyrill Tkachov > wrote: >> Hi all, >> >> I've encountered another wrong-code bug with the store merging pass. This >> time it's in RTL. >> The test gcc.target/aarch64/aapcs64/test_27.c on aarch64 merges a few

Re: [PATCH][simplify-rtx] Zero-initialise local array in simplify_immed_subreg

2016-10-07 Thread Andrew Pinski
On Fri, Oct 7, 2016 at 7:08 AM, Kyrill Tkachov wrote: > Hi all, > > I've encountered another wrong-code bug with the store merging pass. This > time it's in RTL. > The test gcc.target/aarch64/aapcs64/test_27.c on aarch64 merges a few __fp16 > values at GIMPLE level but > during RTL dse1 one of the

Re: [PATCH][simplify-rtx] Zero-initialise local array in simplify_immed_subreg

2016-10-07 Thread Bernd Schmidt
On 10/07/2016 04:08 PM, Kyrill Tkachov wrote: * simplify-rtx.c (simplify_immed_subreg): Zero-initialize tmp array before merging in bytes to pass down to real_from_target. Ok. Bernd

[PATCH][simplify-rtx] Zero-initialise local array in simplify_immed_subreg

2016-10-07 Thread Kyrill Tkachov
Hi all, I've encountered another wrong-code bug with the store merging pass. This time it's in RTL. The test gcc.target/aarch64/aapcs64/test_27.c on aarch64 merges a few __fp16 values at GIMPLE level but during RTL dse1 one of the constants generated gets wrongly misinterpreted from HImode to