On Mon, Oct 14, 2019 at 10:22 PM Jeff Law <l...@redhat.com> wrote: > > 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 <joze...@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 <joze...@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.
Thanks, fixed and applied. Jozef > > > OK with that change. > > jeff
