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.

Reply via email to