Re: Patch drop: fwprop.c patch for neon-strided-load-extract
On 29 December 2011 10:21, Richard Sandiford wrote: > The remaining change for neon-strided-load-extract is to allow fwprop.c > to propagate: > > (set (reg X) (subreg (reg Y) N)) > > even if no further simplifications are possible. I posted the original > patch for comments here: > > http://article.gmane.org/gmane.comp.gcc.patches/246180/ > > and fixed the problem that H.J. spotted. I wasn't entirely happy with > the benchmark results though, so it never became an RFA. When you say you weren't happy with the benchmark results - would you remember if it didn't show any significant improvements or were there significant regressions ? Ramana > > Richard > > > gcc/ > * fwprop.c (propagate_rtx): Also set PR_CAN_APPEAR for subregs. > > Index: gcc/fwprop.c > === > --- gcc/fwprop.c 2011-09-15 14:36:23.206143787 +0100 > +++ gcc/fwprop.c 2011-09-15 14:36:40.995131564 +0100 > @@ -664,7 +664,12 @@ propagate_rtx (rtx x, enum machine_mode > return NULL_RTX; > > flags = 0; > - if (REG_P (new_rtx) || CONSTANT_P (new_rtx)) > + if (REG_P (new_rtx) > + || CONSTANT_P (new_rtx) > + || (GET_CODE (new_rtx) == SUBREG > + && REG_P (SUBREG_REG (new_rtx)) > + && (GET_MODE_SIZE (mode) > + <= GET_MODE_SIZE (GET_MODE (SUBREG_REG (new_rtx)) > flags |= PR_CAN_APPEAR; > if (!for_each_rtx (&new_rtx, varying_mem_p, NULL)) > flags |= PR_HANDLE_MEM; > > ___ > linaro-toolchain mailing list > linaro-toolchain@lists.linaro.org > http://lists.linaro.org/mailman/listinfo/linaro-toolchain ___ linaro-toolchain mailing list linaro-toolchain@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-toolchain
Re: Patch drop: fwprop.c patch for neon-strided-load-extract
Ramana Radhakrishnan writes: > On 29 December 2011 10:21, Richard Sandiford > wrote: >> The remaining change for neon-strided-load-extract is to allow fwprop.c >> to propagate: >> >> (set (reg X) (subreg (reg Y) N)) >> >> even if no further simplifications are possible. I posted the original >> patch for comments here: >> >> http://article.gmane.org/gmane.comp.gcc.patches/246180/ >> >> and fixed the problem that H.J. spotted. I wasn't entirely happy with >> the benchmark results though, so it never became an RFA. > > When you say you weren't happy with the benchmark results - would you > remember if it didn't show > any significant improvements or were there significant regressions ? There were some nice improvements but a couple of regressions. I don't remember the regressions being very big, but they were outside the noise, so I wanted to explain them first. Richard ___ linaro-toolchain mailing list linaro-toolchain@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-toolchain
Re: Patch drop: fwprop.c patch for neon-strided-load-extract
On 4 January 2012 15:09, Richard Sandiford wrote: > Ramana Radhakrishnan writes: >> On 29 December 2011 10:21, Richard Sandiford >> wrote: >>> The remaining change for neon-strided-load-extract is to allow fwprop.c >>> to propagate: >>> >>> (set (reg X) (subreg (reg Y) N)) >>> >>> even if no further simplifications are possible. I posted the original >>> patch for comments here: >>> >>> http://article.gmane.org/gmane.comp.gcc.patches/246180/ >>> >>> and fixed the problem that H.J. spotted. I wasn't entirely happy with >>> the benchmark results though, so it never became an RFA. >> >> When you say you weren't happy with the benchmark results - would you >> remember if it didn't show >> any significant improvements or were there significant regressions ? > > There were some nice improvements but a couple of regressions. I don't > remember the regressions being very big, but they were outside the noise, > so I wanted to explain them first. Ok - thanks that's good enough to know. Ramana > > Richard ___ linaro-toolchain mailing list linaro-toolchain@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-toolchain
[ACTIVITY] weekly status
Hi, * Android * migrated my linaro android build environment * did a small change to the debuggerd patch (thanks Sylvain) * OpenEmbedded * the linaro binary toolchain uses multiarch paths while OE doesn't -> setup a workaround to make it look like a classic one * however, I think what I really want is to build the libc instead of using the one provided by the binary external toolchain * The core-image-minimal still doesn't boot properly because 'init' doesn't come properly Regards Ken ___ linaro-toolchain mailing list linaro-toolchain@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-toolchain
Release notes for 2012.01
I liked Andrew's idea of sharing the release notes so I stole it for this month: https://wiki.linaro.org/MichaelHope/Sandbox/GCC2012.01ReleaseNotes Please add anything new and noteworthy that you've done to this draft. -- Michael ___ linaro-toolchain mailing list linaro-toolchain@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-toolchain
