Re: [PATCH, i386] Remove use of vpmacsdql instruction from multiplication.

2014-08-11 Thread Uros Bizjak
On Tue, Aug 12, 2014 at 7:23 AM, Gopalasubramanian, Ganesh wrote: >> > +2014-06-10 Ganesh Gopalasubramanian >> > + >> > + >> > + * config/i386/i386.c (ix86_expand_sse2_mulvxdi3): Issue >> > +instructions "vpmuludq" and "vpaddq" instead of "vpmacsdql" for >> > +handling 32-bit multiplicatio

RE: [PATCH, i386] Remove use of vpmacsdql instruction from multiplication.

2014-08-11 Thread Gopalasubramanian, Ganesh
Hi Uros! > > +2014-06-10 Ganesh Gopalasubramanian > > + > > + > > + * config/i386/i386.c (ix86_expand_sse2_mulvxdi3): Issue > > +instructions "vpmuludq" and "vpaddq" instead of "vpmacsdql" for > > +handling 32-bit multiplication. > > > OK for mainline and release branches. I would like

Re: [PATCH, i386] Remove use of vpmacsdql instruction from multiplication.

2014-06-10 Thread Uros Bizjak
On Tue, Jun 10, 2014 at 12:30 PM, Gopalasubramanian, Ganesh wrote: > Hi, > > The below patch fixes the issue with 64-bit multiplication. > The instruction "vpmacsdql" does signed 32-bit multiplication. > For V2DImode, we require widened unsigned multiplication. > So, replacing the "vpmacsdql" inst

[PATCH, i386] Remove use of vpmacsdql instruction from multiplication.

2014-06-10 Thread Gopalasubramanian, Ganesh
Hi, The below patch fixes the issue with 64-bit multiplication. The instruction "vpmacsdql" does signed 32-bit multiplication. For V2DImode, we require widened unsigned multiplication. So, replacing the "vpmacsdql" instruction with "vpmuludq" and "vpaddq". This patch had been already discussed in