Re: [PATCH] PR c++/17494 - Fix evaluation of method calls under EVAL_SKIP

[James Greenhalgh <james.greenhalgh@arm.com>]

On Wed, Nov 05, 2014 at 12:24:16AM +0000, Alan Modra wrote:
> On Mon, Nov 03, 2014 at 06:08:15PM -0800, Siva Chandra wrote:
> > >>        PR c++/17494
> > >>        * eval.c (evaluate_subexp_standard): Evaluate the "object" and
> > >>        the method args also under EVAL_SKIP when evaluating method
> > >>        calls under EVAL_SKIP.
> > 
> > Thanks for the review. Pushed: e0f52461c2467b6610391681fa27cd9b3c5def57
> 
> After this patch, when using gcc-4.9.1 and gcc-5.0 (20140911) I see
> what looks to me to be nonsense errors (ie. I'd suspect a compiler
> bug).  Does anyone else see the same?  Nothing special in configure
> options, x86_64-linux.

For what it is worth, I'm seeing the same with my GCC 4.9.1 (x84_64-linux),
again nothing special in the configure options for GCC:

   ../gcc-4.9.1/configure --enable-checking=release --with-cpu=native 

Thanks,
James

> /src/binutils-gdb/gdb/eval.c: In function ‘evaluate_subexp_standard’:
> /src/binutils-gdb/gdb/eval.c:745:16: error: ‘ret’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
>   struct value *ret = NULL;
>                 ^
> In file included from /src/binutils-gdb/gdb/common/common-defs.h:49:0,
>                  from /src/binutils-gdb/gdb/defs.h:28,
>                  from /src/binutils-gdb/gdb/eval.c:20:
> /src/binutils-gdb/gdb/common/common-exceptions.h:148:20: error: ‘buf’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
>         SIGJMP_BUF *buf = \
>                     ^
> /src/binutils-gdb/gdb/common/common-exceptions.h:148:20: note: ‘buf’ was declared here
>         SIGJMP_BUF *buf = \
>                     ^
> /src/binutils-gdb/gdb/eval.c:747:2: note: in expansion of macro ‘TRY_CATCH’
>   TRY_CATCH (ex, RETURN_MASK_ERROR)
>   ^
> /src/binutils-gdb/gdb/eval.c:1429:19: error: ‘value’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
>      struct value *value = NULL;
>                    ^
> In file included from /src/binutils-gdb/gdb/common/common-defs.h:49:0,
>                  from /src/binutils-gdb/gdb/defs.h:28,
>                  from /src/binutils-gdb/gdb/eval.c:20:
> /src/binutils-gdb/gdb/common/common-exceptions.h:148:20: error: ‘buf’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
>         SIGJMP_BUF *buf = \
>                     ^
> /src/binutils-gdb/gdb/common/common-exceptions.h:148:20: note: ‘buf’ was declared here
>         SIGJMP_BUF *buf = \
>                     ^
> /src/binutils-gdb/gdb/eval.c:1430:5: note: in expansion of macro ‘TRY_CATCH’
>      TRY_CATCH (except, RETURN_MASK_ERROR)
>      ^
> /src/binutils-gdb/gdb/eval.c:1846:18: error: ‘value’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
>     struct value *value = NULL;
>                   ^
> In file included from /src/binutils-gdb/gdb/common/common-defs.h:49:0,
>                  from /src/binutils-gdb/gdb/defs.h:28,
>                  from /src/binutils-gdb/gdb/eval.c:20:
> /src/binutils-gdb/gdb/common/common-exceptions.h:148:20: error: ‘buf’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
>         SIGJMP_BUF *buf = \
>                     ^
> /src/binutils-gdb/gdb/common/common-exceptions.h:148:20: note: ‘buf’ was declared here
>         SIGJMP_BUF *buf = \
>                     ^
> /src/binutils-gdb/gdb/eval.c:1847:4: note: in expansion of macro ‘TRY_CATCH’
>     TRY_CATCH (except, RETURN_MASK_ERROR)
>     ^
> cc1: all warnings being treated as errors
> 
> 
> -- 
> Alan Modra
> Australia Development Lab, IBM
>