This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: patch to fix 2 testsuite failures for LRA on PPC
- From: Alan Modra <amodra at gmail dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>
- Cc: David Edelsohn <dje dot gcc at gmail dot com>, Michael Meissner <meissner at linux dot vnet dot ibm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Peter Bergner <bergner at vnet dot ibm dot com>
- Date: Sat, 7 Dec 2013 11:23:00 +1030
- Subject: Re: RFA: patch to fix 2 testsuite failures for LRA on PPC
- Authentication-results: sourceware.org; auth=none
- References: <52A0BA81 dot 2030108 at redhat dot com> <20131206162845 dot GA14285 at ibm-tiger dot the-meissners dot org> <52A209AA dot 7070409 at redhat dot com> <52A21F61 dot 5080308 at redhat dot com> <CAGWvny=w8q=aLo0E+VTRXSXBD909spLHGnB08uZ6dZp6yANA9Q at mail dot gmail dot com> <52A24E60 dot 3040906 at redhat dot com>
On Fri, Dec 06, 2013 at 05:23:28PM -0500, Vladimir Makarov wrote:
> On 12/6/2013, 2:40 PM, David Edelsohn wrote:
> >On Fri, Dec 6, 2013 at 2:02 PM, Vladimir Makarov <vmakarov@redhat.com> wrote:
> >> * config/rs6000/rs600.md (*bswapdi2_64bit): Remove ?? from the
> >> constraint.
> >
> >Okay, let's just remove the "??" modifier from the constraint.
> >
> >Thanks for your patience, explanations, and work on this, Vlad.
> >
>
> Thanks, David.
>
> Committed as rev. 205765.
/* -m32 -O2 -S -mlra */
long long swap64 (long long x)
{
return __builtin_bswap64 (x);
}
Here too, I think. OK to apply David?
* config/rs6000/rs600.md (bswapdi2_32bit): Remove ?? from the
constraint.
Index: gcc/config/rs6000/rs6000.md
===================================================================
--- gcc/config/rs6000/rs6000.md (revision 205767)
+++ gcc/config/rs6000/rs6000.md (working copy)
@@ -2544,7 +2544,7 @@
}")
(define_insn "bswapdi2_32bit"
- [(set (match_operand:DI 0 "reg_or_mem_operand" "=&r,Z,??&r")
+ [(set (match_operand:DI 0 "reg_or_mem_operand" "=&r,Z,&r")
(bswap:DI (match_operand:DI 1 "reg_or_mem_operand" "Z,r,r")))
(clobber (match_scratch:SI 2 "=&b,&b,X"))]
"!TARGET_POWERPC64 && (REG_P (operands[0]) || REG_P (operands[1]))"
--
Alan Modra
Australia Development Lab, IBM