This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 0/5] some combine patches
- From: Oleg Endo <oleg dot endo at t-online dot de>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sun, 16 Nov 2014 17:45:06 +0900
- Subject: Re: [PATCH 0/5] some combine patches
- Authentication-results: sourceware.org; auth=none
- References: <cover dot 1415984897 dot git dot segher at kernel dot crashing dot org>
Hi Segher,
On 15 Nov 2014, at 04:19, Segher Boessenkool <segher@kernel.crashing.org> wrote:
> Here are five patches that together allow combine to do more useful
> work with PARALLELs of two SETs, like on many machines a set of a GPR
> and one of the condition code, or a GPR and the carry bit on PowerPC,
> or two GPRs on some machines.
>
> The first patch is just for debug.
>
> The second is the real meat: it allows combining an I2 that has two SETs.
>
> The third adds a regno field to LOG_LINKS, which the fourth then uses in
> distribute_log_links; without that, most parallels lose their log_links
> early.
>
> The fifth removes a SET from the combination result if it is (now) dead,
> if what's left is a valid instruction.
>
> Bootstrapped and tested on powerpc64-linux (tree of a week ago), all five
> together, -m64,-m32,-m32/-mpowerpc64,-m64/-mlra; no regressions. Checks
> of the separate patches still running. Is this okay for mainline if it
> passes?
When you commit those, could you please also add PR 59278 to the ChangeLog so that the commit appears in bugzilla? After your patches are in, I'd like to add some SH specific test cases (assuming that your patches fix PR 59278).
Cheers,
Oleg