This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][PR target/80083][7 regression] fix power9 vsx-small-integer issue caused by wrong constraints
- From: Aaron Sawdey <acsawdey at linux dot vnet dot ibm dot com>
- To: gcc-patches at gcc dot gnu dot org
- Cc: Segher Boessenkool <segher at kernel dot crashing dot org>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>, meissner at gcc dot gnu dot org, dje dot gcc at gmail dot com
- Date: Mon, 20 Mar 2017 13:05:15 -0500
- Subject: Re: [PATCH][PR target/80083][7 regression] fix power9 vsx-small-integer issue caused by wrong constraints
- Authentication-results: sourceware.org; auth=none
- References: <1490026293.5885.6.camel@linux.vnet.ibm.com>
On Mon, 2017-03-20 at 11:11 -0500, Aaron Sawdey wrote:
> Test libgomp doacross2.f90 failed only at -O1 because an incorrect
> constraint on movsi_internal1 (for vspltisw) led to confusion between
> vsx and float registers (fix credit to Meissner). In subsequent
> discussion David Edelsohn pointed out that there was an additional
> error on the constraint for xxspltib -1 that is also fixed now.
>
> Bootstrap/regtest reveals no errors on either power8 or power9. Ok
> for
> trunk?
>
> 2017-03-20 Aaron Sawdey <acsawdey@linux.vnet.ibm.com>
>
> PR target/80083
> * config/rs6000/rs6000.md (*movsi_internal1): incorrect
> constraints
> for alternatives 14/15.
>
Forgot the patch!
--
Aaron Sawdey, Ph.D. acsawdey@linux.vnet.ibm.com
050-2/C113 (507) 253-7520 home: 507/263-0782
IBM Linux Technology Center - PPC Toolchain
Index: gcc/config/rs6000/rs6000.md
===================================================================
--- gcc/config/rs6000/rs6000.md (revision 246224)
+++ gcc/config/rs6000/rs6000.md (working copy)
@@ -6727,7 +6727,7 @@
[(set (match_operand:SI 0 "rs6000_nonimmediate_operand"
"=r, r, r, ?*wI, ?*wH,
m, ?Z, ?Z, r, r,
- r, ?*wIwH, ?*wJwK, ?*wK, ?*wJwK,
+ r, ?*wIwH, ?*wJwK, ?*wJwK, ?*wu,
?*wJwK, ?*wH, ?*wK, ?*wIwH, ?r,
r, *c*l, *h, *h")