This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: i386 FP comparsion bug (and sollution)
- To: law at cygnus dot com
- Subject: Re: i386 FP comparsion bug (and sollution)
- From: Jan Hubicka <hubicka at atrey dot karlin dot mff dot cuni dot cz>
- Date: Sat, 31 Oct 1998 18:15:49 +0100
- Cc: egcs at cygnus dot com
- References: <19981030102448.60179@atrey.karlin.mff.cuni.cz> <10615.909853654@hurl.cygnus.com>
>
> In message <19981030102448.60179@atrey.karlin.mff.cuni.cz>you write:
> > Yes, it is bug. But why do not turn it to feature?
> No. Not separating the cc0 from the cc0 user is a fundamental design concept,
> we are not changing it anytime soon.
>
> We need to stop regstack from inserting insns between the cc0 setter and
> the cc0 user. Anything else is unacceptable.
OK
I've already read the reg-stack code and I believe I see the resonable solution.
(I should make compare code to emit fst itself (it already support it for fcompp
case and fcomp case is handlel specially in reg-stack.c, so it is probably not so bad
to add yet another exception I wanted to avoid))
I will try to make real fix today. The autor of reg-stack really knew that he is
adding code after condition so I expected it is design decision. Maybe it was
feature but it got forgotten and thats why this bug was created.
Making condition code to handle fst manually is probably better.
At the monday I will also make new version of my third patch w/o this fix
and send it separately. Maybe this fix should be resonable for 1.1.x too...
Honza
>
> jeff
--
OK. Lets make a signature file.
+-------------------------------------------------------------------------+
| Jan Hubicka (Jan Hubi\v{c}ka in TeX) hubicka@freesoft.cz |
| Czech free software foundation: http://www.freesoft.cz |
|AA project - the new way for computer graphics - http://www.ta.jcu.cz/aa |
| homepage: http://www.paru.cas.cz/~hubicka/, games koules, Xonix, fast |
| fractal zoomer XaoS, index of Czech GNU/Linux/UN*X documentation etc. |
+-------------------------------------------------------------------------+