Re: [PATCH] Fix vec_set_hi* patterns (PR target/70059)

2016-03-04 Thread Kirill Yukhin
On 03 Mar 21:17, Jakub Jelinek wrote: > On Thu, Mar 03, 2016 at 01:08:41PM +0100, Jakub Jelinek wrote: > > Fixed thusly, unfortunately I don't have access to avx512f (and not even to > > avx512dq) hw, so while I will bootstrap/regtest it on Haswell-E, can't test > > the tests if they now work at ru

Re: [PATCH] Fix vec_set_hi* patterns (PR target/70059)

2016-03-03 Thread Jakub Jelinek
On Thu, Mar 03, 2016 at 01:08:41PM +0100, Jakub Jelinek wrote: > Fixed thusly, unfortunately I don't have access to avx512f (and not even to > avx512dq) hw, so while I will bootstrap/regtest it on Haswell-E, can't test > the tests if they now work at runtime (they link and the assembly of the foo >

Re: [PATCH] Fix vec_set_hi* patterns (PR target/70059)

2016-03-03 Thread Kirill Yukhin
Hi Jakub, On 03 Mar 13:08, Jakub Jelinek wrote: > routine has changed and looks good to me). Can somebody test this please > on real hw or emulator? I'll run testing on the simulator. -- Thanks, K

[PATCH] Fix vec_set_hi* patterns (PR target/70059)

2016-03-03 Thread Jakub Jelinek
Hi! Unlike the older vec_set_hi 256-bit patterns, which are correctly using the VEC_SELECT as the first operand of VEC_CONCAT and match_operand as second, because that is what the instruction does, picks up low part from operand 1 and puts the operand 2 as the high part of the result, the 512-bit