base,scons: `-wno-deprecated-copy` added for hdf5.cc with GCC
authorBobby R. Bruce <bbruce@ucdavis.edu>
Fri, 24 Jul 2020 01:22:09 +0000 (18:22 -0700)
committerBobby R. Bruce <bbruce@ucdavis.edu>
Wed, 29 Jul 2020 00:35:00 +0000 (00:35 +0000)
As highlighted by Ciro here:
https://gem5-review.googlesource.com/c/public/gem5/+/31216, and
here: https://gem5.atlassian.net/browse/GEM5-365,  It appears that GCC
versions >= 9 requires `-wno-deprecated-copy` which was removed in
commit: https://gem5-review.googlesource.com/c/public/gem5/+/31216.
`-wno-deprecated-copy` appears to work for all versions of GCC. Clang
does not require `-wno-deprecated-copy` nor `-wno-deprecated` for
sucessfull compilation. Therefore branching has been introduced to the
SConscript to address this and simplify the solution.

Change-Id: I233b32aa945d479dd429bb5591272608ba342d8d
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/31754
Reviewed-by: Bobby R. Bruce <bbruce@ucdavis.edu>
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>
Tested-by: kokoro <noreply+kokoro@google.com>
src/base/SConscript

index 657528c94e413503fda55881995861a5e8372e33..b7a0c1b28998214bf00f1f228667baacff39edf6 100644 (file)
@@ -91,7 +91,10 @@ Source('loader/symtab.cc')
 Source('stats/group.cc')
 Source('stats/text.cc')
 if env['USE_HDF5']:
-    Source('stats/hdf5.cc', append={'CXXFLAGS': '-Wno-deprecated'})
+    if main['GCC']:
+        Source('stats/hdf5.cc', append={'CXXFLAGS': '-Wno-deprecated-copy'})
+    else:
+        Source('stats/hdf5.cc')
 
 GTest('addr_range.test', 'addr_range.test.cc')
 GTest('addr_range_map.test', 'addr_range_map.test.cc')