From patchwork Tue Mar 13 19:14:20 2012
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [i386] RTM support
Date: Tue, 13 Mar 2012 09:14:20 -0000
From: Uros Bizjak <ubizjak@gmail.com>
X-Patchwork-Id: 146471
Message-Id: <CAFULd4YYdpv+fpXj2K7wSPU3Fszr7-XvfF+sUJuxGMjLEat8fg@mail.gmail.com>
To: Kirill Yukhin <kirill.yukhin@gmail.com>
Cc: Jakub Jelinek <jakub@redhat.com>,
 gcc-patches List <gcc-patches@gcc.gnu.org>,
 "H.J. Lu" <hjl.tools@gmail.com>, Andi Kleen <andi@firstfloor.org>,
 Richard Henderson <rth@redhat.com>,
 Tobias Burnus <tobias.burnus@physik.fu-berlin.de>

On Sun, Mar 11, 2012 at 10:16 AM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
>>
>> The patch is OK for mainline, if there are no further comments in next 24h.
>
> According to Tobias's input, I've added few lines about RTM to
> doc/invoke.texi. If no objections - I'll commit the patch tomorrow.

A small no-op change - there is no need for a constraint in an expand
pattern.  Plus some formatting.

2012-03-13  Uros Bizjak  <ubizjak@gmail.com>

	* config/i386/i386.md (xbegin): Remove constraint from expander.

Tested on x86_64-pc-linux-gnu, committed to mainline SVN.

Uros.


Index: i386.md
===================================================================
--- i386.md     (revision 185350)
+++ i386.md     (working copy)
@@ -18206,7 +18206,7 @@
    (set_attr "memory" "unknown")])

 (define_expand "xbegin"
-  [(set (match_operand:SI 0 "register_operand" "=a")
+  [(set (match_operand:SI 0 "register_operand" "")
        (unspec_volatile:SI [(match_dup 1)] UNSPECV_XBEGIN))]
   "TARGET_RTM"
 {
@@ -18258,8 +18258,8 @@
 {
   emit_insn (gen_xtest_1 ());

-  ix86_expand_setcc (operands[0], EQ, gen_rtx_REG (CCZmode,
FLAGS_REG), const0_rtx);
-
+  ix86_expand_setcc (operands[0], EQ,
+                    gen_rtx_REG (CCZmode, FLAGS_REG), const0_rtx);
   DONE;
 })