Thanks!
K
On Tue, Mar 13, 2012 at 11:14 PM, Uros Bizjak <ubiz...@gmail.com> wrote:
> On Sun, Mar 11, 2012 at 10:16 AM, Kirill Yukhin <kirill.yuk...@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 <ubiz...@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;
> })
