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: ARM: fix generation of widening multiply on v6+

From: Zack Weinberg <zack at codesourcery dot com>
To: Paul Brook <paul at codesourcery dot com>
Cc: gcc-patches at gcc dot gnu dot org
Date: Mon, 03 Sep 2007 16:59:56 -0700
Subject: Re: ARM: fix generation of widening multiply on v6+
References: <46DC5665.2000806@codesourcery.com> <200709040057.11707.paul@codesourcery.com>

Paul Brook wrote:

	* config/arm/arm.md (mulsidi3, umulsidi3, mulsi3_highpart)
	(umulsi3_highpart): Make into expanders; existing insns are
	now named *[insn]_nov6.  Add stars to existing [insn]_v6
	counterparts' names.

+(define_expand "umulsi3_highpart"
...
+     (clobber (match_scratch:SI 3 "=&r,&r"))])]

Constraints aren't meaningful in an expander. You removed them from smulsi3_highpart, so I assume this is an oversight.

Yup. Will fix.

zw

References:
- ARM: fix generation of widening multiply on v6+
  - From: Zack Weinberg
- Re: ARM: fix generation of widening multiply on v6+
  - From: Paul Brook

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