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 PR37053: Move tweaks of commutative precedence to target hook


Luis Machado wrote:
Hi folks,

Hi Luis,


There are two patches at the moment: the long one (adding target hook) and the short one (using reload_in_progress and reload_completed).


I gave it a try with Maxim's patch from ticket #37053 and it degraded performance pretty badly (> 30%), and galgel degraded more than 75%. Both 32-bit and 64-bit saw degradations.

The patch in PR37053 is the one adding target hook. This patch should not affect the generated code at all. If it does, can you please send the the difference in dumps? I understand that this patch does not seem to be a favorite here, but I'm curious where I've made a mistake.



Worth noticing that the patch posted here didn't degrade performance (i heard that's Paolo's idea):

http://gcc.gnu.org/ml/gcc-patches/2009-06/msg02038.html

This is the short patch and it's good news it didn't degrade.


--
Maxim


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