Review Board 2.0.15


scons: Move the warning about self assignment to SWIG code only

Review Request #1938 - Created June 28, 2013 and discarded

Information
Andreas Hansson
gem5
default
Reviewers
Default
Changeset 9802:5523750ec6fe
---------------------------
scons: Move the warning about self assignment to SWIG code only

This patch changes the gcc and clang flags such that the
self-assignment warning is no longer globally disabled, and moves it
to the SWIG flags. The instances where the warning causes problems are
addressed as part of the patch.

Note that this patch causes plenty warnings about uninitialised
values, as we are currently hiding this with "int a; a = a;"
constructs. Anyone brave enough (and skilled in the arts) should have
a look and fix these. This affects X86, MIPS and SPARC. ALPHA and ARM
work without any issues.
ALPHA and ARM regressions pass, the other ISAs fail to compile due to uninitialised variables. Help to figure out what to do with these would be much appreciated.
Review request changed
Updated (July 5, 2013, 6:31 a.m.)

Status: Discarded