Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-25 Thread Ulrich Drepper
On Mon, Mar 24, 2014 at 9:09 AM, Jakub Jelinek wrote: > The following is recognized well: > > typedef char v32qi __attribute__((vector_size (32))); > v32qi foo (char a) > { > return (v32qi) { a, a, a, a, a, a, a, a, a, a, a, a, a, a, a, a, a, a, a, > a, a, a, a, a, a, a, a, a, a, a, a, a }; > }

Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-24 Thread Uros Bizjak
Hello! > Another set of functions missing are those to set all elements of a > 512-bit vector to the same float or double value. I think the patch > below uses the optimal code sequence for that. The patch requires the > previous patch introducing _mm*_undefined_*. > > > 2014-03-19 Ulrich Drepp

Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-24 Thread Jakub Jelinek
On Mon, Mar 24, 2014 at 01:33:11PM +0100, Richard Biener wrote: > On Mon, Mar 24, 2014 at 1:25 PM, Uros Bizjak wrote: > > Hello! > > > >> On Mon, Mar 24, 2014 at 12:13 PM, Ulrich Drepper wrote: > Your patch is correct IMHO, but maybe it worst to add all missing > `mm512_set1*' stuff? >

Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-24 Thread Richard Biener
On Mon, Mar 24, 2014 at 1:25 PM, Uros Bizjak wrote: > Hello! > >> On Mon, Mar 24, 2014 at 12:13 PM, Ulrich Drepper wrote: Your patch is correct IMHO, but maybe it worst to add all missing `mm512_set1*' stuff? According to trunk and [1] we're still missing (beside mentioned by

Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-24 Thread Uros Bizjak
Hello! > On Mon, Mar 24, 2014 at 12:13 PM, Ulrich Drepper wrote: >>> Your patch is correct IMHO, but maybe it worst to add all missing >>> `mm512_set1*' stuff? >>> >>> According to trunk and [1] we're still missing (beside mentioned by you) >>> _mm512_set1_epi16 and _mm512_set1_epi8 broadcasts.

Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-24 Thread Richard Biener
On Mon, Mar 24, 2014 at 12:13 PM, Ulrich Drepper wrote: > On Mon, Mar 24, 2014 at 1:50 AM, Kirill Yukhin > wrote: >> Your patch is correct IMHO, but maybe it worst to add all missing >> `mm512_set1*' stuff? >> >> According to trunk and [1] we're still missing (beside mentioned by you) >> _mm512_

Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-24 Thread Ulrich Drepper
On Mon, Mar 24, 2014 at 1:50 AM, Kirill Yukhin wrote: > Your patch is correct IMHO, but maybe it worst to add all missing > `mm512_set1*' stuff? > > According to trunk and [1] we're still missing (beside mentioned by you) > _mm512_set1_epi16 and _mm512_set1_epi8 broadcasts. Yes, more are missing

Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-23 Thread Kirill Yukhin
Hello Ulrich, On 19 Mar 22:41, Ulrich Drepper wrote: > Another set of functions missing are those to set all elements of a > 512-bit vector to the same float or double value. I think the patch > below uses the optimal code sequence for that. The patch requires the > previous patch introducing _mm