This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH][AARCH64]Fix PR63424 by adding <su><maxmin>v2di3 pattern

From: Renlin Li <renlin dot li at arm dot com>
To: Richard Henderson <rth at redhat dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Cc: Marcus Shawcroft <Marcus dot Shawcroft at arm dot com>, Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>
Date: Wed, 19 Nov 2014 16:39:44 +0000
Subject: Re: [PATCH][AARCH64]Fix PR63424 by adding <su><maxmin>v2di3 pattern
Authentication-results: sourceware.org; auth=none
References: <5453A1E8 dot 1070003 at arm dot com> <546C7CF7 dot 7010206 at redhat dot com>

On 19/11/14 11:20, Richard Henderson wrote:

On 10/31/2014 03:51 PM, Renlin Li wrote:

+(define_expand "<su><maxmin>v2di3"
+  [(parallel [
+    (set (match_operand:V2DI 0 "register_operand" "")
+	 (MAXMIN:V2DI (match_operand:V2DI 1 "register_operand" "")
+		  (match_operand:V2DI 2 "register_operand" "")))
+    (clobber (reg:CC CC_REGNUM))])]
+  "TARGET_SIMD"

There's no clobber of CC_REGNUM, so you can take that out.

Otherwise it looks good.


r~

committed with your suggestion.

https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=217786

Regards,
Renlin Li

References:
- Re: [PATCH][AARCH64]Fix PR63424 by adding <su><maxmin>v2di3 pattern
  - From: Richard Henderson

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]