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: GCC 3.1 Issues



>>> I have two PRs that might be related:
>>> - PR 5172: A miscompilation of glibc on i686, this is fixed if store
>>>   motion is disable.
>>
>> Ok for branch then?
>
> That's the obvious workaround.  I would prefer if somebody fixes store
> motion correctly (but it might be that this cannot be done on the
> release branch) - and if nobody likes to do this, we should consider
> removing it again.

May I draw the following recent thread to people's attention:

[PATCH] Fix PR opt/5172: Store motion vs. asm_operands
http://gcc.gnu.org/ml/gcc-patches/2002-01/msg01142.html

In the review of that patch, Richard Henderson determined that it was
the original code in glibc that was at fault, for not correctly making
the memory reference explicit in the __asm__ statement.  I guess no-one
updated its GNATS entry to reflect this.

The other PR may or may not actually be broken, but it should be should
be investigated rather than disable store-motion in 3.1 if its actually
working correctly.

Roger
--
Roger Sayle,                         E-mail: roger@eyesopen.com
OpenEye Scientific Software,         WWW: http://www.eyesopen.com/
Suite 1107, 3600 Cerrillos Road,     Tel: (+1) 505-473-7385
Santa Fe, New Mexico, 87507.         Fax: (+1) 505-473-0833


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