On 06/22/2016 06:37 AM, Andi Kleen wrote:
From: Andi Kleen <a...@linux.intel.com>

Extend the existing bprob and tree-prof tests to also run with autofdo.
The test runtimes are really a bit too short for autofdo, but it's
a reasonable sanity check.

This only works natively for now.

dejagnu doesn't seem to support a wrapper for unix tests, so I had
to open code running these tests.  That should be ok due to the
native run restrictions.

gcc/testsuite/:

2016-06-22  Andi Kleen  <a...@linux.intel.com>

        * g++.dg/bprob/bprob.exp: Support autofdo.
        * g++.dg/tree-prof/tree-prof.exp: dito.
        * gcc.dg/tree-prof/tree-prof.exp: dito.
        * gcc.misc-tests/bprob.exp: dito.
        * gfortran.dg/prof/prof.exp: dito.
        * lib/profopt.exp: dito.
        * lib/target-supports.exp: Check for autofdo.
I'm generally OK with this as well. My only concern is that we get something sensible on targets which don't support autofdo and on systems without perf installed.

ISTM the right result is for the autofdo versions of those tests to be UNSUPPORTED on both those situations. Can you confirm that's what happens in those two cases? If we do indeed get UNSUPPORTED for the autofdo versions of those tests in those cases, then this patch is OK as well.

jeff

Reply via email to