public inbox for gcc-prs@sourceware.org
help / color / mirror / Atom feed
* Re: optimization/8178: __builtin_ffs broken
@ 2003-03-12 12:36 Volker Reichelt
0 siblings, 0 replies; 6+ messages in thread
From: Volker Reichelt @ 2003-03-12 12:36 UTC (permalink / raw)
To: nobody; +Cc: gcc-prs
The following reply was made to PR optimization/8178; it has been noted by GNATS.
From: Volker Reichelt <reichelt@igpm.rwth-aachen.de>
To: christianbiere@gmx.de, gcc-gnats@gcc.gnu.org, gcc-bugs@gcc.gnu.org,
rth@redhat.com, falk.hueffner@student.uni-tuebingen.de
Cc: ebotcazou@libertysurf.fr
Subject: Re: optimization/8178: __builtin_ffs broken
Date: Wed, 12 Mar 2003 13:30:29 +0100 (CET)
I can confirm that the problem is still present on k6.
According to Phil's tester the problem for the ix86 was fixed
between 2003-01-31 and 2003-02-01.
Therefore I strongly suspect that the following patch (by Richard
Henderson and Falk Hueffner) was responsible for the fix:
http://gcc.gnu.org/ml/gcc-cvs/2003-02/msg00033.html
That's why I'm asking you, Richard and Falk, could you please
have a look at the patch, whether it really did fix things and if yes,
why it doesn't fix the k6-issue and whether the fix can be backported
to 3.3 or even 3.2?
Thanks,
Volker
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: optimization/8178: __builtin_ffs broken
@ 2003-04-18 16:26 Falk Hueffner
0 siblings, 0 replies; 6+ messages in thread
From: Falk Hueffner @ 2003-04-18 16:26 UTC (permalink / raw)
To: nobody; +Cc: gcc-prs
The following reply was made to PR optimization/8178; it has been noted by GNATS.
From: Falk Hueffner <falk.hueffner@student.uni-tuebingen.de>
To: reichelt@igpm.rwth-aachen.de
Cc: christianbiere@gmx.de, gcc-bugs@gcc.gnu.org, gcc-prs@gcc.gnu.org,
nobody@gcc.gnu.org, gcc-gnats@gcc.gnu.org
Subject: Re: optimization/8178: __builtin_ffs broken
Date: 18 Apr 2003 18:23:15 +0200
reichelt@igpm.rwth-aachen.de writes:
> Synopsis: __builtin_ffs broken
>
> State-Changed-From-To: analyzed->closed
> State-Changed-By: reichelt
> State-Changed-When: Fri Apr 18 15:36:52 2003
> State-Changed-Why:
> Fixed in gcc 3.2 branch, 3.3 branch and mainline.
It still doesn't work for me on mainline with -march=k6:
% cat ffs.c
int f(int x) { return __builtin_ffs(x); }
% /tmp/bla/bin/gcc -v
Reading specs from /usr/tmp/bla/bin/../lib/gcc-lib/i586-pc-linux-gnu/3.4/specs
Configured with: ./configure --enable-languages=c --disable-nls
Thread model: posix
gcc version 3.4 20030415 (experimental)
% /tmp/bla/bin/gcc -c ffs.c
% /tmp/bla/bin/gcc -c -march=k6 ffs.c
ffs.c: In function `f':
ffs.c:1: error: unrecognizable insn:
(insn 26 22 27 0 0x401705d8 (parallel [
(set (reg:SI 1 edx)
(const_int 0 [0x0]))
(clobber (reg:CC 17 flags))
]) -1 (nil)
(nil))
ffs.c:1: internal compiler error: in extract_insn, at recog.c:2188
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.
--
Falk
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: optimization/8178: __builtin_ffs broken
@ 2003-04-18 15:36 reichelt
0 siblings, 0 replies; 6+ messages in thread
From: reichelt @ 2003-04-18 15:36 UTC (permalink / raw)
To: christianbiere, gcc-bugs, gcc-prs, nobody
Synopsis: __builtin_ffs broken
State-Changed-From-To: analyzed->closed
State-Changed-By: reichelt
State-Changed-When: Fri Apr 18 15:36:52 2003
State-Changed-Why:
Fixed in gcc 3.2 branch, 3.3 branch and mainline.
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=8178
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: optimization/8178: __builtin_ffs broken
@ 2003-03-13 0:46 Richard Henderson
0 siblings, 0 replies; 6+ messages in thread
From: Richard Henderson @ 2003-03-13 0:46 UTC (permalink / raw)
To: nobody; +Cc: gcc-prs
The following reply was made to PR optimization/8178; it has been noted by GNATS.
From: Richard Henderson <rth@redhat.com>
To: Volker Reichelt <reichelt@igpm.rwth-aachen.de>
Cc: christianbiere@gmx.de, gcc-gnats@gcc.gnu.org, gcc-bugs@gcc.gnu.org,
falk.hueffner@student.uni-tuebingen.de, ebotcazou@libertysurf.fr,
gcc-patches@gcc.gnu.org
Subject: Re: optimization/8178: __builtin_ffs broken
Date: Wed, 12 Mar 2003 16:42:15 -0800
On Wed, Mar 12, 2003 at 01:30:29PM +0100, Volker Reichelt wrote:
> I can confirm that the problem is still present on k6.
> According to Phil's tester the problem for the ix86 was fixed
> between 2003-01-31 and 2003-02-01.
Backporting the patch you mention isn't an option, but
this bug is fixable for 3.2/3.3 trivially.
r~
* config/i386/i386.md (ffssi2): Tighten op1 predicate to
match the requirements of the output insns.
Index: gcc/config/i386/i386.md
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/i386/i386.md,v
retrieving revision 1.404.2.10
diff -c -p -d -u -r1.404.2.10 i386.md
--- gcc/config/i386/i386.md 12 Mar 2003 09:35:27 -0000 1.404.2.10
+++ gcc/config/i386/i386.md 13 Mar 2003 00:41:08 -0000
@@ -13671,7 +13671,7 @@
\f
(define_expand "ffssi2"
[(set (match_operand:SI 0 "nonimmediate_operand" "")
- (ffs:SI (match_operand:SI 1 "general_operand" "")))]
+ (ffs:SI (match_operand:SI 1 "nonimmediate_operand" "")))]
""
{
rtx out = gen_reg_rtx (SImode), tmp = gen_reg_rtx (SImode);
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: optimization/8178: __builtin_ffs broken
@ 2003-03-10 15:46 Eric Botcazou
0 siblings, 0 replies; 6+ messages in thread
From: Eric Botcazou @ 2003-03-10 15:46 UTC (permalink / raw)
To: nobody; +Cc: gcc-prs
The following reply was made to PR optimization/8178; it has been noted by GNATS.
From: Eric Botcazou <ebotcazou@libertysurf.fr>
To: Volker Reichelt <reichelt@igpm.rwth-aachen.de>
Cc: hristianbiere@gmx.de,
gcc-gnats@gcc.gnu.org,
gcc-bugs@gcc.gnu.org
Subject: Re: optimization/8178: __builtin_ffs broken
Date: Mon, 10 Mar 2003 16:41:50 +0100
> The problem got apparently fixed on mainline.
Probably by http://gcc.gnu.org/ml/gcc-patches/2003-02/msg00300.html
> I'm leaving the PR open, since somebody might want to backport the fix
> to the 3.3 or 3.2 branch.
I'm still getting an ICE on k6 though:
http://gcc.gnu.org/ml/gcc-patches/2003-03/msg00707.html
--
Eric Botcazou
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: optimization/8178: __builtin_ffs broken
@ 2003-03-10 15:06 Volker Reichelt
0 siblings, 0 replies; 6+ messages in thread
From: Volker Reichelt @ 2003-03-10 15:06 UTC (permalink / raw)
To: nobody; +Cc: gcc-prs
The following reply was made to PR optimization/8178; it has been noted by GNATS.
From: Volker Reichelt <reichelt@igpm.rwth-aachen.de>
To: christianbiere@gmx.de, gcc-gnats@gcc.gnu.org, gcc-bugs@gcc.gnu.org
Cc:
Subject: Re: optimization/8178: __builtin_ffs broken
Date: Mon, 10 Mar 2003 16:01:52 +0100 (CET)
The problem got apparently fixed on mainline.
I'm leaving the PR open, since somebody might want to backport the fix
to the 3.3 or 3.2 branch.
Regards,
Volker
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=8178
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2003-04-18 16:26 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-03-12 12:36 optimization/8178: __builtin_ffs broken Volker Reichelt
-- strict thread matches above, loose matches on Subject: below --
2003-04-18 16:26 Falk Hueffner
2003-04-18 15:36 reichelt
2003-03-13 0:46 Richard Henderson
2003-03-10 15:46 Eric Botcazou
2003-03-10 15:06 Volker Reichelt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).