This is the mail archive of the gcc@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]

Re: i386 FP comparsion bug (and sollution)


> 
>   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.  | 
+-------------------------------------------------------------------------+


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