On 07/25/2011 07:11 PM, Tom de Vries wrote: > Hi, > > thanks for the review. > > On 07/18/2011 03:19 PM, Richard Earnshaw wrote: >> On 18/07/11 12:09, Tom de Vries wrote: >>> Hi, >>> >>> PR43597 was fixed by >>> http://gcc.gnu.org/viewcvs?view=revision&revision=172032. >>> >>> This patch adds a testcase. >>> >>> OK for trunk? >>> >>> Thanks, >>> - Tom >>> >>> 2011-07-18 Tom de Vries <t...@codesourcery.com> >>> >>> PR target/43597 >>> * gcc.target/arm/pr43597.c: New test. >>> >>> >> >> No, don't pass -mthumb through dg-options unless you're using something >> like require-effective-target. >> > > OK, I see. > >> In this case the post-compile tests are all gated on thumb2. So why not >> make the whole test just require arm_thumb2_ok? >> >> R. >> > > Done. OK for trunk? >
Ping. http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02173.html OK for trunk? Thanks, -Tom > 2011-07-25 Tom de Vries <t...@codesourcery.com> > > PR target/43597 > * gcc.target/arm/pr43597.c: New test.