This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [PATCH] splits after reload
- From: "Rakesh Kumar - Software, Noida" <rakeshku at noida dot hcltech dot com>
- To: Joern Rennecke <joern dot rennecke at superh dot com>, rth at redhat dot com
- Cc: "Rakesh Kumar - Software, Noida" <rakeshku at noida dot hcltech dot com>, gcc-patches at gcc dot gnu dot org
- Date: Thu, 5 Feb 2004 17:17:58 +0530
- Subject: RE: [PATCH] splits after reload
> -----Original Message-----
> From: gcc-patches-owner@gcc.gnu.org
> [mailto:gcc-patches-owner@gcc.gnu.org]On Behalf Of Joern Rennecke
> Sent: Wednesday, February 04, 2004 8:18 PM
> To: rth@redhat.com
> Cc: rakeshku@noida.hcltech.com; gcc-patches@gcc.gnu.org;
> joern.rennecke@superh.com
> Subject: Re: [PATCH] splits after reload
> If reload_cse does the job remains subject to benchmarking. But even
> if it didn't, a new, separate pass would certainly be
> cleaner, and pathologic
> time behaviour can be better avoided or rectified by
> appropriate algorithms
> and data strctures.
The problem I'm talking about is acute on SH. Please have
a look at the following thread.
http://gcc.gnu.org/ml/gcc/2003-12/msg00700.html
To avoid compile time overhead for other targets, a target hook
looked reasonable. Other targets can make use of that hook
(if some are facing similar problems) and solve their "target specific
problem in a target specific way".
Running a pass over all insns generated by the splitter looks expensive.
It sounds like if SH needs some optimization it has to pay "tax"
going through all the condition checks which are bound to be
the part of generic optimizer.
reload_cse is not able to perform this optimization, even if
you run it after flow2. There is no code written in reload_cse for handling
such problems. Please correct me if I'm wrong.
If reload_cse () ought to do so,
I feel that needs rewriting of reload_cse which certainly is not a
trivial task. The target hook should be acceptable in the meantime.
I agree that recombination is not the purpose of split_all_insns ().
But a target hook helps without harming any other target.
Please have a look at the patch and let me know what is wrong with it.
I'm thankful to you for your valuable time and suggestions.
Regards,
Rakesh Kumar