Re: [PR87054] fix unaligned access

2018-09-24 Thread Richard Biener
On Mon, 24 Sep 2018, Alexandre Oliva wrote: > On Sep 18, 2018, Richard Biener wrote: > > >> +# ifdef __SSE__ > >> +# define T __int128 > > Rainer reported this didn't work on 32-bit x86 with SSE enabled. > Here's a patch that fixes it. Ok to install? Tested on x86_64, > including such combin

Re: [PR87054] fix unaligned access

2018-09-24 Thread Alexandre Oliva
On Sep 18, 2018, Richard Biener wrote: >> +# ifdef __SSE__ >> +# define T __int128 Rainer reported this didn't work on 32-bit x86 with SSE enabled. Here's a patch that fixes it. Ok to install? Tested on x86_64, including such combinations as -m32, -m32/-msse, and -m32/-mno-sse. [PR87054] ad

Re: [PR87054] fix unaligned access

2018-09-17 Thread Richard Biener
On September 18, 2018 5:47:34 AM GMT+02:00, Alexandre Oliva wrote: >Richard, this is the patch you proposed in the PR a while ago, along >with the testcase I'd posted. Please let me know if the commit-message >paragraph below sounds good to you, and then I'll proceed to install >it, >or amend as

[PR87054] fix unaligned access

2018-09-17 Thread Alexandre Oliva
Richard, this is the patch you proposed in the PR a while ago, along with the testcase I'd posted. Please let me know if the commit-message paragraph below sounds good to you, and then I'll proceed to install it, or amend as requested. (do we really want the MEM_REF to carry stricter alignment th