This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: long long performance
- To: Torbjorn Granlund <tege at swox dot com>
- Subject: Re: long long performance
- From: Richard Henderson <rth at redhat dot com>
- Date: Thu, 14 Dec 2000 00:15:10 -0800
- Cc: Rob Willis <rob at e-critical dot com>, gcc at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- References: <3A37F1ED.ED6C4133@e-critical.com> <86puivq4hj.fsf@king.swox.se>
On Thu, Dec 14, 2000 at 12:35:36AM +0100, Torbjorn Granlund wrote:
> I think the * should go away from
>
> (define_insn "*umulsi3_highpart"
> (define_insn "*smulsi3_highpart"
>
> so that the code generator can use these!
I agree. Looking at the logs, it appears that these were
"commented out" by mistake during some over-eager pattern
cleanups back in January.
r~
* config/i386/i386.md (umulsi3_highpart): Remove * from name.
(smulsi3_highpart): Likewise.
Index: i386.md
===================================================================
RCS file: /cvs/gcc/egcs/gcc/config/i386/i386.md,v
retrieving revision 1.195
diff -c -p -d -r1.195 i386.md
*** i386.md 2000/12/10 19:51:07 1.195
--- i386.md 2000/12/14 08:10:39
***************
*** 5061,5067 ****
(set_attr "length_immediate" "0")
(set_attr "mode" "SI")])
! (define_insn "*umulsi3_highpart"
[(set (match_operand:SI 0 "register_operand" "=d")
(truncate:SI
(lshiftrt:DI
--- 5061,5067 ----
(set_attr "length_immediate" "0")
(set_attr "mode" "SI")])
! (define_insn "umulsi3_highpart"
[(set (match_operand:SI 0 "register_operand" "=d")
(truncate:SI
(lshiftrt:DI
***************
*** 5079,5085 ****
(set_attr "length_immediate" "0")
(set_attr "mode" "SI")])
! (define_insn "*smulsi3_highpart"
[(set (match_operand:SI 0 "register_operand" "=d")
(truncate:SI
(lshiftrt:DI
--- 5079,5085 ----
(set_attr "length_immediate" "0")
(set_attr "mode" "SI")])
! (define_insn "smulsi3_highpart"
[(set (match_operand:SI 0 "register_operand" "=d")
(truncate:SI
(lshiftrt:DI