This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Alias changes
- To: Mark Mitchell <mark at markmitchell dot com>
- Subject: Re: Alias changes
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Thu, 25 Jun 1998 00:29:50 -0600
- Cc: egcs-patches at cygnus dot com
- Reply-To: law at cygnus dot com
I'd prefer to have this option off by default for the short term; just
call me Mr. Conservative. I'll personally be trying it, but I'm a little
uncomfortable having it be on by default for the upcoming release.
Somewhere in the rtl/tree changes you should note that the alias set
is just an integer and does not directly correspond to a particular
type, storage class, memory location, etc. That's a pretty fundamental
concept that needs a mention somewhere (if I missed it, then ignore my
comment).
I think we need to save/restore the current alias set # as we change
function contexts so that the current alias set # isn't scrogged
when we compile a nested function. Even if this isn't an issue right
now we need to go ahead and save/restore the set #. It's simple to
do, so I don't think you need to resubmit after making this change.
We've already discussed the long vs int stuff.
I think you should install the patch after making the changes noted
above.
jeff