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 PR middle-end/28690, indexed load/store performance + reload bug



And in fact I realized that if you fix LEGITIMIZE_MEMORY_ADDRESS, you
probably don't need this patch anyhow.

I am not sure. Actually, I am pretty sure the canonicalization which is the basic idea of this patch pays off on other platforms. But to do it properly, we first need to make sure GCC does not needlessly pessimize code that, when canonicalized this way, would undo the other canonicalization in swap_commutative_operands_with_target (for example). Then we can do much more, for example by boosting the precedence of REGs vs. MEMs (something that right now pays off IIRC on PowerPC but -- I'm sure about this -- loses on x86).


Paolo


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