On Wednesday 01 June 2016 16:33:30 Kyrill Tkachov wrote: > I see gcc-patches was not cc'ed on the original submission and I didn't CC > it myself in my reply, so here it is a resend with my reply... > > On 31/05/16 14:11, Thomas Preudhomme wrote: > > Hi, > > > > sync_char_short and sync_int_long used to restrict platform running > > gcc.dg/atomic* tests are only true for ARM when targetting Linux. > > However, > > ARMv8-A brought atomic instructions support so these tests could be run > > on > > bare metal ARMv8-A targets. > > > > This patch define these two effective target to be true for ARM when the > > platform supports lda* and stl* instructions. > > > > ChangeLog entry is as follows: > > > > > > *** gcc/ChangeLog *** > > > > 2016-05-26 Thomas Preud'homme > > > > * doc/sourcebuild.texi (arm_acq_rel): Document new effective > > target. > > > > *** gcc/testsuite/ChangeLog *** > > > > 2016-05-26 Thomas Preud'homme > > > > * lib/target-supports.exp (check_effective_target_arm_acq_rel): > > New > > procedure. > > Have you confirmed that the tests that now become enabled > for bare-metal pass on an arm-none-eabi test run with -march=armv8-a? I did but had a doubt about whether I also tried with -march=armv8-a -mthumb so I tried that just in case and it's all good. > If so, this is ok with the typo fixes below. Please resend the patch > you intend to commit to gcc-patches for archival purposes. Thanks. Committed the patch in attachment. Best regards, Thomas