[Patch][i386] PR 70118: Fix ubsan warning on SSE2 loadl_epi64 and storel_epi64

2016-11-26 Thread Allan Sandfeld Jensen
Use the recently introduced unaligned variant of __m128i and add a similar __m64 and use those to make it clear these two intrinsics require neither 128- bit nor 64-bit alignment. `Allan Index: gcc/config/i386/emmintrin.h === --- gcc

Re: [Patch][i386] PR 70118: Fix ubsan warning on SSE2 loadl_epi64 and storel_epi64

2016-11-26 Thread Marc Glisse
On Sat, 26 Nov 2016, Allan Sandfeld Jensen wrote: Use the recently introduced unaligned variant of __m128i and add a similar __m64 and use those to make it clear these two intrinsics require neither 128- bit nor 64-bit alignment. Thanks for doing this. You'll want Uros or Kirill to review your

Re: [Patch][i386] PR 70118: Fix ubsan warning on SSE2 loadl_epi64 and storel_epi64

2016-11-27 Thread Allan Sandfeld Jensen
On Sunday 27 November 2016, Marc Glisse wrote: > On Sat, 26 Nov 2016, Allan Sandfeld Jensen wrote: > > Use the recently introduced unaligned variant of __m128i and add a > > similar __m64 and use those to make it clear these two intrinsics > > require neither 128- bit nor 64-bit alignment. > > Tha

Re: [Patch][i386] PR 70118: Fix ubsan warning on SSE2 loadl_epi64 and storel_epi64

2016-12-05 Thread Allan Sandfeld Jensen
Trying again, this time with changelog. gcc/ 2016-11-30 Allan Sandfeld Jensen PR target/70118 * gcc/config/i386/mmintrin.h (__m64_u): New type * gcc/config/i386/emmintrin.h (_mm_loadl_epi64, _mm_storel_epi64): Make the allowed unaligned memory access explici

Re: [Patch][i386] PR 70118: Fix ubsan warning on SSE2 loadl_epi64 and storel_epi64

2016-12-06 Thread Uros Bizjak
Hello! > 2016-11-30 Allan Sandfeld Jensen > >PR target/70118 >* gcc/config/i386/mmintrin.h (__m64_u): New type >* gcc/config/i386/emmintrin.h (_mm_loadl_epi64, _mm_storel_epi64): >Make the allowed unaligned memory access explicit. OK. Thanks, Uros.

Re: [Patch][i386] PR 70118: Fix ubsan warning on SSE2 loadl_epi64 and storel_epi64

2016-12-09 Thread Allan Sandfeld Jensen
On Tuesday 06 December 2016, Uros Bizjak wrote: > Hello! > > > 2016-11-30 Allan Sandfeld Jensen > > > >PR target/70118 > >* gcc/config/i386/mmintrin.h (__m64_u): New type > >* gcc/config/i386/emmintrin.h (_mm_loadl_epi64, _mm_storel_epi64): > >Make the allowed u

Re: [Patch][i386] PR 70118: Fix ubsan warning on SSE2 loadl_epi64 and storel_epi64

2016-12-11 Thread Uros Bizjak
On Fri, Dec 9, 2016 at 11:49 PM, Allan Sandfeld Jensen wrote: > On Tuesday 06 December 2016, Uros Bizjak wrote: >> Hello! >> >> > 2016-11-30 Allan Sandfeld Jensen >> > >> >PR target/70118 >> >* gcc/config/i386/mmintrin.h (__m64_u): New type >> >* gcc/config/i386/emmintri