misc: Updates for gcc7.2 for x86
authorJason Lowe-Power <jason@lowepower.com>
Wed, 13 Dec 2017 18:19:04 +0000 (10:19 -0800)
committerJason Lowe-Power <jason@lowepower.com>
Thu, 14 Dec 2017 00:27:59 +0000 (00:27 +0000)
commit5c41076bd7610d03431fd0dd89bd0fdc7f30d6bd
treeee24cdf5ea368fb808d0b6029313a163c375898b
parentf07d5069d86e31ecf195664850f79fb00c445bd3
misc: Updates for gcc7.2 for x86

GCC 7.2 is much stricter than previous GCC versions. The following changes
are needed:

* There is now a warning if there is an implicit fallthrough between two
  case statments. C++17 adds the [[fallthrough]]; declaration. However,
  to support non C++17 standards (i.e., C++11), we use M5_FALLTHROUGH.
  M5_FALLTHROUGH checks for [[fallthrough]] compliant C++17 compiler and
  if that doesn't exist, it defaults to nothing (no older compilers
  generate warnings).
* The above resulted in a couple of bugs that were found. This is noted
  in the review request on gerrit.
* throw() for dynamic exception specification is deprecated
* There were a couple of new uninitialized variable warnings
* Can no longer perform bitwise operations on a bool.
* Must now include <functional> for std::function
* Compiler bug for void* lambda. Changed to auto as work around. See
  https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82878

Change-Id: I5d4c782a4e133fa4cdb119e35d9aff68c6e2958e
Signed-off-by: Jason Lowe-Power <jason@lowepower.com>
Reviewed-on: https://gem5-review.googlesource.com/5802
Reviewed-by: Gabe Black <gabeblack@google.com>
21 files changed:
ext/dnet/ip.h
ext/drampower/src/Utils.h
src/arch/arm/isa.cc
src/arch/arm/table_walker.cc
src/arch/x86/isa.cc
src/arch/x86/isa/microops/regop.isa
src/base/compiler.hh
src/base/cprintf.cc
src/base/imgwriter.cc
src/cpu/kvm/base.cc
src/cpu/kvm/x86_cpu.cc
src/cpu/minor/dyn_inst.cc
src/dev/arm/generic_timer.cc
src/dev/net/i8254xGBe.cc
src/dev/pci/copy_engine.cc
src/dev/storage/ide_disk.cc
src/dev/x86/i8042.cc
src/kern/linux/printk.cc
src/mem/slicc/symbols/Type.py
src/sim/eventq.hh
src/sim/fd_array.cc