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: small varasm.c improvements


David Edelsohn <dje@watson.ibm.com> writes:

> 	I believe the issue is that we've swapped the special pool entry
> for the original symbol, so there is nothing to prevent the original
> symbol from possibly being deleted because the special pool entry has been
> substituted everywhere.  The USE makes sure that the symbol being
> referenced by the pool is not itself deleted.  Geoff may have some ideas
> about whether this is fixable, or maybe we just XFAIL it on rs6000 port.

The right thing to do is to give varasm.c more information about
what's going on and what refers to what; then the SET will be
unnecessary.  In fact, ideally all the constant pool manipulation that
the rs6000 backend does should be in varasm.c; all it's really trying
to do is have a single constant pool for the entire file rather than
one per function, and refer to all the constants through a pointer table.

-- 
- Geoffrey Keating <geoffk@geoffk.org>


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