Patch to improve PIC find_base_term

John Wehle john@feith.com
Mon Aug 21 23:29:00 GMT 2000


> I'm very surprised by this.  All of the memory loads from the GOT
> should already be marked as RTX_UNCHANGING_P, which should take 
> care of everything wrt aliasing.  What is slipping through?

The change to find_base_term wasn't primarily for memory loads from
the GOT ... the performance difference is probably due to other
aliasing issues addressed by the patch (which I'm re-examining /
reworking based on everyone's feedback).

I did apply the suggestion from Geoff regarding assigning an alias
set for loads from the GOT and there does seem to be a minor
performance difference.  Currently alias analysis assumes that
an unchanging write and an unchanging read may conflict, unless
there's information to the contrary so maybe that's what is
"slipping through" ... I don't actually know.  I've copied
you on the patch.

-- John
-------------------------------------------------------------------------
|   Feith Systems  |   Voice: 1-215-646-8000  |  Email: john@feith.com  |
|    John Wehle    |     Fax: 1-215-540-5495  |                         |
-------------------------------------------------------------------------



More information about the Gcc-patches mailing list