On Mon, Apr 1, 2019 at 12:49 AM CHIGOT, CLEMENT <clement.chi...@atos.net> wrote: > > Description: > * This patches disables go tests with -flto on AIX. > > Tests: > * AIX 7.2: Configure/Build: SUCCESS > > Changelog: > * lib/go-torture.exp: add check for lto option
Thanks. I moved the adjustment up into the block where TORTURE_OPTIONS was not already set, and committed as follows. Ian 2019-04-03 Clément Chigot <clement.chi...@atos.net> * lib/go-torture.exp: Only add lto to TORTURE_OPTIONS if it is supported.
Index: lib/go-torture.exp =================================================================== --- lib/go-torture.exp (revision 269805) +++ lib/go-torture.exp (working copy) @@ -34,10 +34,13 @@ if ![info exists TORTURE_OPTIONS] { { -O2 -fomit-frame-pointer -finline-functions -funroll-loops } \ { -O2 -fbounds-check } \ { -O3 -g } \ - { -Os } \ - { -flto }] -} + { -Os }] + if [check_effective_target_lto] { + set TORTURE_OPTIONS \ + [concat $TORTURE_OPTIONS [list {-flto}]] + } +} # # go-torture-compile -- compile a go.go-torture testcase.