On Nov 19, 2016, at 1:59 PM, Andrew Burgess <andrew.burg...@embecosm.com> wrote: >> So, your new test fails on arm* targets: > > After a little digging I think the problem might be that > -freorder-blocks-and-partition is not supported on arm. > > This should be detected as the new tests include: > > /* { dg-require-effective-target freorder } */ > > however this test passed on arm as -freorder-blocks-and-partition does > not issue any warning unless -fprofile-use is also passed. > > The patch below extends check_effective_target_freorder to check using > -fprofile-use. With this change in place the tests are skipped on > arm.
> All feedback welcome, Seems reasonable, unless a -freorder-blocks-and-partition/-fprofile-use person thinks this is the wrong solution.