On Wed, May 03, 2017 at 11:20:24PM -0400, Christopher Li wrote:
> On Wed, May 3, 2017 at 10:14 PM, Luc Van Oostenryck
> <luc.vanoostenryck@xxxxxxxxx> wrote:
> >
> > Yes, but there are a few details we need to look at.
> > For example, this should be supported (GCC does)
> > int foo(a)
> > register int a;
> > {
> > return a;
> > }
> >
> > But this we should warn on this
> > struct s {
> > register int a;
> > };
> > And currently, we don't.
>
> You are right. If we really want to fix that, we could add some field into
> "struct decl_state" to indicate we are in the struct mode, some storage
> specifier does not apply.
>
> Can be a separate patch.
It's not that I care much about pre-ANSI syntax but this 'register' allowed
inside a struct really bugs me.
I'll look for a patchs in the coming days.
-- Luc
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
