Hi Uros, Checked the patch in the following branches and trunk after bootstrapping and regression testing them individually.
GCC 5 branch https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=224215 GCC 4.9 branch https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=224214 GCC 4.8 branch https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=224147 Trunk https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=224146 regards, Venkat. > -----Original Message----- > From: Kumar, Venkataramanan > Sent: Thursday, June 04, 2015 8:44 PM > To: Uros Bizjak (ubiz...@gmail.com); gcc-patches@gcc.gnu.org > Subject: [Patch] [X86_64]: fix operand constraints in sse3_mwait > > Hi Uros, > > As discussed here https://gcc.gnu.org/ml/gcc/2015-06/msg00043.html > > I am going to install the following patch to trunk. > > GCC bootstrap and regressions tests passed. > > Regards, > Venkat. > > diff --git a/gcc/ChangeLog b/gcc/ChangeLog index ab5c004..2fa6e96 100644 > --- a/gcc/ChangeLog > +++ b/gcc/ChangeLog > @@ -1,3 +1,7 @@ > +2015-06-04 Venkataramanan Kumar <Venkataramanan.kumar> > + > + * config/i386/sse.md (sse3_mwait): Swap the operand constraints. > + > 2015-06-02 Alan Modra <amo...@gmail.com> > > * config/rs6000/vsx.md (vsx_extract_v4sf): Revert accidental diff > --git > a/gcc/config/i386/sse.md b/gcc/config/i386/sse.md index 21c6c6c..2685f06 > 100644 > --- a/gcc/config/i386/sse.md > +++ b/gcc/config/i386/sse.md > @@ -13194,10 +13194,12 @@ > (set_attr "atom_sse_attr" "fence") > (set_attr "memory" "unknown")]) > > - > +;; As per AMD and Intel ISA manuals, the first operand is extensions ;; > +and it goes to %ecx. The second operand received is hints and it goes > +;; to %eax. > (define_insn "sse3_mwait" > - [(unspec_volatile [(match_operand:SI 0 "register_operand" "a") > - (match_operand:SI 1 "register_operand" "c")] > + [(unspec_volatile [(match_operand:SI 0 "register_operand" "c") > + (match_operand:SI 1 "register_operand" "a")] > UNSPECV_MWAIT)] > "TARGET_SSE3" > ;; 64bit version is "mwait %rax,%rcx". But only lower 32bits are used. >
