Re: [PATCH i386 6/8] [AVX-512] Add builtins/intrinsics.

2013-12-30 Thread Kirill Yukhin
Hello Uroš, On 18 Dec 18:16, Uros Bizjak wrote: > +/* Walk through insns sequence or pattern and erase rounding mentions. > + Main transformation is performed in ix86_erase_embedded_rounding_1. */ > +static rtx > +ix86_erase_embedded_rounding (rtx pat) > > All calls to this function are made wi

Re: [PATCH i386 6/8] [AVX-512] Add builtins/intrinsics.

2013-12-27 Thread Uros Bizjak
On Fri, Dec 27, 2013 at 4:04 PM, Kirill Yukhin wrote: > On 18 Dec 18:16, Uros Bizjak wrote: >> the patch is OK (with above mentioned changes) for mainline. > Thanks, > > One more nit. It seems that currently vectorizer expects mask_type to > be equal to operand_type, which doesn't hold for AVX-51

Re: [PATCH i386 6/8] [AVX-512] Add builtins/intrinsics.

2013-12-27 Thread Kirill Yukhin
Hello, On 18 Dec 18:16, Uros Bizjak wrote: > the patch is OK (with above mentioned changes) for mainline. Thanks, One more nit. It seems that currently vectorizer expects mask_type to be equal to operand_type, which doesn't hold for AVX-512F. So, I'll comment out like this: @@ -34677,6 +36423,3

Re: [PATCH i386 6/8] [AVX-512] Add builtins/intrinsics.

2013-12-18 Thread Uros Bizjak
On Wed, Dec 18, 2013 at 2:21 PM, Kirill Yukhin wrote: > Hello, > > On 02 Dec 16:15, Kirill Yukhin wrote: >> Hello >> > Ok for trunk? >> Ping? > Ping. > > Rebased patch attached. + error ("third argument must be comparison constant."); "the third ...", without dot at the end. Please review m

Re: [PATCH i386 6/8] [AVX-512] Add builtins/intrinsics.

2013-12-18 Thread Kirill Yukhin
Hello, On 02 Dec 16:15, Kirill Yukhin wrote: > Hello > > Ok for trunk? > Ping? Ping. Rebased patch attached. Thanks, K p.patch.bz2 Description: BZip2 compressed data

Re: [PATCH i386 6/8] [AVX-512] Add builtins/intrinsics.

2013-12-02 Thread Kirill Yukhin
Hello > Ok for trunk? Ping? -- Thanks, K