[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-08-22 Thread hubicka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #14 from Jan Hubicka --- The testcase still seems to work for me (with and without linker plugin). Without linker plugin collect2 will not execute LTO unless it can resolve symbols. You need to link with g++ (and have runtime or path

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-08-22 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #13 from Manuel López-Ibáñez --- (In reply to Jan Hubicka from comment #12) > > So my tree shows it by default. I am at a conference and have to go for > talks, I cannot reproduce the above in trunk. Is it that a testcase that only

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread hubicka at ucw dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #12 from Jan Hubicka --- > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 > > --- Comment #6 from Manuel López-Ibá??ez --- > (In reply to Marek Polacek from comment #5) > > (In reply to Manuel López-Ibá??ez from comment #4) > >

Re: [Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread Jan Hubicka
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 > > --- Comment #6 from Manuel López-Ibá??ez --- > (In reply to Marek Polacek from comment #5) > > (In reply to Manuel López-Ibá??ez from comment #4) > > > It is pretty strange that you need a Var() if it is not used anywhere. > > > > It is, w

Re: [Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread Jan Hubicka
> > It seems quite a major deficiency if we are going to keep adding warnings to > > LTO, which are never tested and there is a high chance they are not working > > at all. Is there a PR about it? Is Jan or someone else working on it? > > I don't know about such PR. Neither do I know whether Jan'

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread hubicka at ucw dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #11 from Jan Hubicka --- > > It seems quite a major deficiency if we are going to keep adding warnings to > > LTO, which are never tested and there is a high chance they are not working > > at all. Is there a PR about it? Is Jan or so

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #10 from Marek Polacek --- (In reply to Manuel López-Ibáñez from comment #9) > It doesn't seem to be the same bug at all. The other one does not ICE, just > ignores -Wno-. I understand why -Wno- is ignored, but I don't understand why

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #9 from Manuel López-Ibáñez --- (In reply to Marek Polacek from comment #8) > (In reply to Manuel López-Ibáñez from comment #6) > > (In reply to Marek Polacek from comment #5) > > > (In reply to Manuel López-Ibáñez from comment #4) >

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #8 from Marek Polacek --- (In reply to Manuel López-Ibáñez from comment #6) > (In reply to Marek Polacek from comment #5) > > (In reply to Manuel López-Ibáñez from comment #4) > > > It is pretty strange that you need a Var() if it is

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #7 from Manuel López-Ibáñez --- (In reply to Marek Polacek from comment #5) > > Also, most of my comments in this review have been ignored: > > > > https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00233.html In particular, the manual d

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #6 from Manuel López-Ibáñez --- (In reply to Marek Polacek from comment #5) > (In reply to Manuel López-Ibáñez from comment #4) > > It is pretty strange that you need a Var() if it is not used anywhere. > > It is, we have a bug for i

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #5 from Marek Polacek --- (In reply to Manuel López-Ibáñez from comment #4) > It is pretty strange that you need a Var() if it is not used anywhere. It is, we have a bug for it: PR61480. > Also, most of my comments in this review ha

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org --- Commen

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 Marek Polacek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-28 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 --- Comment #2 from Marek Polacek --- Author: mpolacek Date: Mon Jul 28 08:33:33 2014 New Revision: 213115 URL: https://gcc.gnu.org/viewcvs?rev=213115&root=gcc&view=rev Log: PR middle-end/61913 * common.opt (Wodr): Add Var. Modified:

[Bug middle-end/61913] ICE in common_handle_option with -Wodr

2014-07-26 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913 Marek Polacek changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|