https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61913

--- Comment #9 from Manuel López-Ibáñez <manu at gcc dot gnu.org> ---
(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)
> > > > It is pretty strange that you need a Var() if it is not used anywhere.
> > > 
> > > It is, we have a bug for it: PR61480.
> > 
> > But there is no Init(1) there. In fact, I doubt Wodr is enabled by default
> > as the documentation says without Init(1).
> 
> I guess it doesn't matter whether there's Init() or not then.

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 it
-Wodr ICEs without Var().

> > > The issue is that lto.exp does not allow to scan for 'warning:'.
> > 
> > Isn't that a bug/missing feature? It seems crucial if one wants to add
> > warnings/errors generated by LTO.
> 
> It certainly is a deficiency.

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?

> > In particular, the manual does not show Wodr:
> > https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#Warning-Options
> 
> The description is there (after -Wno-overflow), but @item is missing, so the
> warning name isn't listed there.  I'll add it.

I added the correct entry in my review. You could use that. It fixes several
deficiencies with the current entry.

Reply via email to