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] move superblock formation to Tree-SSA


Robert,
thanks for bringing tracer to SSA lavel.  The implementation seems sane
to me (modulo the problem mentioned by Zdenek) and missing comments in
the front of new functions required by coding standard.
I wonder if you did some actual benchmarks comparing old and new tracer
behaviour?  The placement in pass queue seems a bit early to me, since
tracer might actually confuse some other optimizations and thus it might
be profitable to move it down in the queue.  It might be even profitable
to retain the current RTL functionality as we do for some other code
duplicating optimizations, such as unroller, so ther later tracer can
duplicate stuff that compiled to small code and was missed by earlier
tracing.

If it is problematic for you to run few benchmarks, I can do SPECs on
AMD64.

Thanks,
Honza


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