scons: Pull Configure() to earlier in SConstruct so we can use it more.
authorGabe Black <gabe.black@gmail.com>
Sun, 7 Feb 2021 09:41:27 +0000 (01:41 -0800)
committerGabe Black <gabe.black@gmail.com>
Wed, 10 Feb 2021 16:57:17 +0000 (16:57 +0000)
This mechanism lets us check if headers are available, flags are
supported, etc. We should use that more often, rather than checking for
specific versions of tools where problematic new warnings were added, etc.

Change-Id: I5b1a6499147f27cc8944fcb8c61eb69e9fa8fb7a
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/40860
Reviewed-by: Gabe Black <gabe.black@gmail.com>
Maintainer: Gabe Black <gabe.black@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
SConstruct

index 1485a36a848868dc0d5a3c207c2b25d4a8543e93..d64842425b26dd83ff586e655d8673ca05e87e4b 100755 (executable)
@@ -296,6 +296,10 @@ main['LTO_LDFLAGS'] = []
 # compiler we're using.
 main['TCMALLOC_CCFLAGS'] = []
 
+# Platform-specific configuration.  Note again that we assume that all
+# builds under a given build root run on the same host platform.
+conf = gem5_scons.Configure(main)
+
 CXX_version = readCommand([main['CXX'],'--version'], exception=False)
 CXX_V = readCommand([main['CXX'],'-V'], exception=False)
 
@@ -499,9 +503,6 @@ except Exception as e:
     warning('While checking protoc version:', str(e))
     main['HAVE_PROTOC'] = False
 
-# Platform-specific configuration.  Note again that we assume that all
-# builds under a given build root run on the same host platform.
-conf = gem5_scons.Configure(main)
 
 # Cache build files in the supplied directory.
 if main['M5_BUILD_CACHE']: