[Bug lto/57602] Runfails for several C/C++ benchmarks from spec2000 for i686 with -flto after r199422

[izamyatin at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57602

Igor Zamyatin <izamyatin at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hubicka at ucw dot cz

--- Comment #3 from Igor Zamyatin <izamyatin at gmail dot com> ---
I believe the problem happens because cleanup that was made in
http://gcc.gnu.org/ml/gcc-patches/2013-05/msg01644.html
sometimes changes field 'analyzed' to 'symbol.definition' but at the same time
in other places to 'symbol.analyzed'.
It looks like 'symbol.definition' corresponds to the former 'finalized' field
not 'analyzed' and 'symbol.definition' are not always the same as
'symbol.analyzed'.

I'm attaching the patch (no bootstrap and make check testing though) which
fixes this (but not in all places across the compiler, sort of minimum
intrusion) and runfails disappeared with these changes.

Jan, could you please take a close look since now many x86 32-bit apps built
with lto are broken.