On Tue, May 17, 2016 at 1:25 PM, Christophe Lyon <christophe.l...@linaro.org> wrote: > On 1 April 2016 at 17:32, Ramana Radhakrishnan > <ramana.radhakrish...@foss.arm.com> wrote: >> I've had this in my tree for a few months now but never got >> around to submitting it. >> >> This partially fixes PR target/53440 atleast in ARM and >> Thumb2 state. I haven't yet managed to get my head around >> rewriting the Thumb1 support yet. >> >> Tested on armhf with a bootstrap and regression test >> with no regressions. >> > > Hi Ramana, > > It took me a while to understand why the test was failing on a Thumb1 target > despite the dg-skip directive. > The problem was that dg-do was after dg-skip. > > I've checked in the swap, I hope it is "obvious" enough.
Thanks - yes that's an obvious fix. Ramana > > Christophe. > >> Queued for stage1 now as it isn't technically a regression. >> >> regards >> Ramana >> >> >> <DATE> Ramana Radhakrishnan <ramana.radhakrish...@arm.com> >> >> PR target/53440 >> * config/arm/arm.c (arm32_output_mi_thunk): New. >> (arm_output_mi_thunk): Rename to arm_thumb1_mi_thunk. Rework >> to split Thumb1 vs TARGET_32BIT functionality. >> (arm_thumb1_mi_thunk): New. >> >> >> * g++.dg/inherit/thunk1.C: Support arm / aarch64.
