This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 2/2][MSP430] Optimize zero_extend insns and PSImode pointer manipulation
- From: Jeff Law <law at redhat dot com>
- To: Jozef Lawrynowicz <jozef dot l at mittosystems dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 14 Oct 2019 15:22:26 -0600
- Subject: Re: [PATCH 2/2][MSP430] Optimize zero_extend insns and PSImode pointer manipulation
- References: <20191008113450.5b6fa184@jozef-kubuntu> <20191008113957.054ec793@jozef-kubuntu>
On 10/8/19 4:39 AM, Jozef Lawrynowicz wrote:
> This patch has the functional changes to optimize zero_extend insns and pointer
> manipulation in the large memory model.
>
>
> 0002-MSP430-PSImode-pointer-manipulation-and-zero-extend-.patch
>
> From f8156e115c4743ce94a86835ffa5601b6d28a555 Mon Sep 17 00:00:00 2001
> From: Jozef Lawrynowicz <jozef.l@mittosystems.com>
> Date: Mon, 7 Oct 2019 11:44:16 +0100
> Subject: [PATCH 2/2] MSP430: PSImode pointer manipulation and zero extend insn
> optimizations
>
> gcc/ChangeLog:
>
> 2019-10-08 Jozef Lawrynowicz <jozef.l@mittosystems.com>
>
> * config/msp430/msp430.md (movqipsi): New.
> (zero_extendqipsi2): New.
> (zero_extendqisi2): Optimize case where src register and base dst
> register are the same.
> (zero_extendhipsi2): Don't use 430X insn for rYs->r case.
> (zero_extendpsisi2): Optimize r->m case.
> Add unnamed insn patterns to catch insns combine searches for when
> optimizing pointer manipulation.
So you've got a movqipsi and zero_extendqipsi2 with the same RTL
structure. ISTM the movqipsi should just get removed.
OK with that change.
jeff