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: [PATCH]: Fix alias grouping of name tags.


On 12/9/06, Eric Botcazou <ebotcazou@libertysurf.fr> wrote:
> It simply changes the order in which the alias list is sorted, which
> has no effect on the actual *answers* alias analysis provides (IE
> virtual operands generated, etc).

If you're 100% sure it cannot have any effect whatsoever on code generation,
I'm fine with that, but then please add a comment stating it for the future
reader.
I'm 100% sure it cannot affect code generation.
I'm not sure why you are asking for this comment.  It sounds an awful
lot like you want me to write "This code actually works and does what
it is supposed to, regardless of what host you run it on", which seems
wrong.  We generally document what we think isn't portable, not what
is portable.
That said, if you want to do it, go ahead.

 Otherwise it probably should not be very hard to make the sorting
function itself independent from the host.

If you want to do this, go ahead too.


You should be able to just subtract DECL_UID's.

Though why you'd want to insert two extra loads for code that is 100%
guaranteed to work as is, i'm not sure.


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