Oops! Sorry for such a stupid problem. 2012-06-18 Joey Ye <joey...@arm.com>
Backported from mainline 2011-10-14 David Alan Gilbert <david.gilb...@linaro.org> * config/arm/arm.h (TARGET_HAVE_DMB_MCR): MCR Not available in Thumb1. Index: gcc/config/arm/arm.h =================================================================== --- gcc/config/arm/arm.h (revision 188331) +++ gcc/config/arm/arm.h (working copy) @@ -294,7 +294,8 @@ #define TARGET_HAVE_DMB (arm_arch7) /* Nonzero if this chip implements a memory barrier via CP15. */ -#define TARGET_HAVE_DMB_MCR (arm_arch6k && ! TARGET_HAVE_DMB) +#define TARGET_HAVE_DMB_MCR (arm_arch6 && ! TARGET_HAVE_DMB \ + && ! TARGET_THUMB1) /* Nonzero if this chip implements a memory barrier instruction. */ #define TARGET_HAVE_MEMORY_BARRIER (TARGET_HAVE_DMB || TARGET_HAVE_DMB_MCR) > -----Original Message----- > From: Richard Earnshaw > Sent: Tuesday, June 19, 2012 16:43 > To: Joey Ye > Cc: GCC Patches > Subject: Re: [4.6][ARM] Backport "MCR Not available in Thumb1" > > On 19/06/12 04:03, Joey Ye wrote: > > Backporting trunk r179979 > > > > OK for 4.6? > > > > Backported from mainline > > 2011-10-14 David Alan Gilbert <david.gilb...@linaro.org> > > > > PR target/48126 > > * config/arm/arm.c (arm_output_sync_loop): Move label before > > barrier. > > > > Index: gcc/config/arm/arm.h > > =================================================================== > > --- gcc/config/arm/arm.h (revision 188331) > > +++ gcc/config/arm/arm.h (working copy) > > @@ -294,7 +294,8 @@ > > #define TARGET_HAVE_DMB (arm_arch7) > > > > /* Nonzero if this chip implements a memory barrier via CP15. */ > > -#define TARGET_HAVE_DMB_MCR (arm_arch6k && ! TARGET_HAVE_DMB) > > +#define TARGET_HAVE_DMB_MCR (arm_arch6 && ! TARGET_HAVE_DMB \ > > + && ! TARGET_THUMB1) > > > > /* Nonzero if this chip implements a memory barrier instruction. */ > > #define TARGET_HAVE_MEMORY_BARRIER (TARGET_HAVE_DMB || > TARGET_HAVE_DMB_MCR) > > > > > > Not ok (yet), the ChangeLog entry doesn't match the patch. > > R.