gcc-patches@gcc.gnu.org
[Top] [All Lists]

Re: [PATCH] Fix PR37053: Move tweaks of commutative precedence to targe

Subject: Re: [PATCH] Fix PR37053: Move tweaks of commutative precedence to target hook
From: Maxim Kuvyrkov
Date: Sat, 27 Jun 2009 20:40:48 +0400
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

<Prev in Thread] Current Thread [Next in Thread>