* RE: [Patch] [X86_64]: fix operand constraints in sse3_mwait
@ 2015-06-08 10:44 Kumar, Venkataramanan
0 siblings, 0 replies; 2+ messages in thread
From: Kumar, Venkataramanan @ 2015-06-08 10:44 UTC (permalink / raw)
To: Uros Bizjak (ubizjak@gmail.com), gcc-patches
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 (ubizjak@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 <amodra@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.
>
^ permalink raw reply [flat|nested] 2+ messages in thread
* [Patch] [X86_64]: fix operand constraints in sse3_mwait
@ 2015-06-04 15:20 Kumar, Venkataramanan
0 siblings, 0 replies; 2+ messages in thread
From: Kumar, Venkataramanan @ 2015-06-04 15:20 UTC (permalink / raw)
To: Uros Bizjak (ubizjak@gmail.com), gcc-patches
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 <amodra@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.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-06-08 10:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-08 10:44 [Patch] [X86_64]: fix operand constraints in sse3_mwait Kumar, Venkataramanan
-- strict thread matches above, loose matches on Subject: below --
2015-06-04 15:20 Kumar, Venkataramanan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).