Re: [PATCH, GCC/ARM/gcc-7-branch] Backport PR71607

2017-06-01 Thread Prakhar Bahuguna
On 01/06/2017 07:15:47, Richard Sandiford wrote: > Prakhar Bahuguna writes: > > On 31/05/2017 14:11:43, Richard Sandiford wrote: > >> Prakhar Bahuguna writes: > >> > On 31/05/2017 09:19:40, Richard Sandiford wrote: > >> >> const_ints are supposed to be stored in sign-extended form, so a 32-bit >

Re: [PATCH, GCC/ARM/gcc-7-branch] Backport PR71607

2017-05-31 Thread Richard Sandiford
Prakhar Bahuguna writes: > On 31/05/2017 14:11:43, Richard Sandiford wrote: >> Prakhar Bahuguna writes: >> > On 31/05/2017 09:19:40, Richard Sandiford wrote: >> >> const_ints are supposed to be stored in sign-extended form, so a 32-bit >> >> integer with the MSB set should be 0x8000|x

Re: [PATCH, GCC/ARM/gcc-7-branch] Backport PR71607

2017-05-31 Thread Prakhar Bahuguna
On 31/05/2017 14:11:43, Richard Sandiford wrote: > Prakhar Bahuguna writes: > > On 31/05/2017 09:19:40, Richard Sandiford wrote: > >> const_ints are supposed to be stored in sign-extended form, so a 32-bit > >> integer with the MSB set should be 0x8000|x instead of > >> 0x8000|x.

Re: [PATCH, GCC/ARM/gcc-7-branch] Backport PR71607

2017-05-31 Thread Richard Sandiford
Prakhar Bahuguna writes: > On 31/05/2017 09:19:40, Richard Sandiford wrote: >> const_ints are supposed to be stored in sign-extended form, so a 32-bit >> integer with the MSB set should be 0x8000|x instead of >> 0x8000|x. It's a bug if you have one where that isn't true. >> >> In

Re: [PATCH, GCC/ARM/gcc-7-branch] Backport PR71607

2017-05-31 Thread Prakhar Bahuguna
On 31/05/2017 09:19:40, Richard Sandiford wrote: > const_ints are supposed to be stored in sign-extended form, so a 32-bit > integer with the MSB set should be 0x8000|x instead of > 0x8000|x. It's a bug if you have one where that isn't true. > > In the patch it looks like this cou

Re: [PATCH, GCC/ARM/gcc-7-branch] Backport PR71607

2017-05-31 Thread Richard Sandiford
Prakhar Bahuguna writes: > This patch tackles the issue reported in PR71607. This patch takes a different > approach for disabling the creation of literal pools. Instead of disabling the > patterns that would normally transform the rtl into actual literal pools, it > disables the creation of this