On Sun, Jul 27, 2014 at 1:02 PM, Marek Polacek pola...@redhat.com wrote:
Wodr in common.opt was missing a Var, which means:
1) we ICE with -Wodr, since -Wodr isn't handled in opts.c;
2) -Wno-odr wouldn't work.
Thus fixed. I'd think this doesn't need a testcase...
Bootstrapped/regtested on
On Sun, Jul 27, 2014 at 1:02 PM, Marek Polacek pola...@redhat.com wrote:
Wodr in common.opt was missing a Var, which means:
1) we ICE with -Wodr, since -Wodr isn't handled in opts.c;
2) -Wno-odr wouldn't work.
Thus fixed. I'd think this doesn't need a testcase...
On Mon, Jul 28, 2014 at 10:23:36AM +0200, Jan Hubicka wrote:
On Sun, Jul 27, 2014 at 1:02 PM, Marek Polacek pola...@redhat.com wrote:
Wodr in common.opt was missing a Var, which means:
1) we ICE with -Wodr, since -Wodr isn't handled in opts.c;
2) -Wno-odr wouldn't work.
Thus fixed.
On Mon, Jul 28, 2014 at 10:23:36AM +0200, Jan Hubicka wrote:
On Sun, Jul 27, 2014 at 1:02 PM, Marek Polacek pola...@redhat.com wrote:
Wodr in common.opt was missing a Var, which means:
1) we ICE with -Wodr, since -Wodr isn't handled in opts.c;
2) -Wno-odr wouldn't work.
Thus
Wodr in common.opt was missing a Var, which means:
1) we ICE with -Wodr, since -Wodr isn't handled in opts.c;
2) -Wno-odr wouldn't work.
Thus fixed. I'd think this doesn't need a testcase...
Bootstrapped/regtested on x86_64-linux, ok for trunk?
2014-07-27 Marek Polacek pola...@redhat.com