On Mon, 17 Sep 2018, 23:56 Christophe Lyon, <christophe.l...@linaro.org> wrote:
> On Fri, 14 Sep 2018 at 12:04, Sam Tebbs <sam.te...@arm.com> wrote: > > > > > > > > On 08/28/2018 11:54 PM, James Greenhalgh wrote: > > > > <snip> > > > > > > OK once the other one is approved, with the obvious rebase over the > renamed > > > function. > > > > > > James > > > > Here is the rebased patch. Still OK for me to commit to trunk now that > > the other patch has been committed? > > > > Sam > > > > > > > >> gcc/ > > >> 2018-07-31 Sam Tebbs <sam.te...@arm.com> > > >> > > >> PR target/85628 > > >> * config/aarch64/aarch64.md (*aarch64_bfxilsi_uxtw): Define. > > >> > > >> gcc/testsuite > > >> 2018-07-31 Sam Tebbs <sam.te...@arm.com> > > >> > > >> PR target/85628 > > >> * gcc.target/aarch64/combine_bfxil.c > > >> (combine_zero_extended_int, foo6): > > >> New functions. > > > > Hi Sam, > > This patch causes a regression when using -mabi=ilp32: > FAIL: gcc.target/aarch64/combine_bfxil.c scan-assembler-not uxtw\\t > > How much do we care about ilp32? > Lets keep the testsuite clean please. Ramana > > Christophe >