[PATCH 0/2] list of gcc supported attribute name

[PATCH 0/2] list of gcc supported attribute name

[Christopher Li]

This patch add full list of known gcc supported attributes.

So sparse should not give warning on the legit gcc attribute
any more.

Test with full kernel allmodeconfig compile with the exact
same output as before(master branch).

I want to include this two patches in the sparse release 0.5.1.

Git repository here:
https://git.kernel.org/pub/scm/devel/sparse/sparse.git/log/?h=sparse-next-20170603

Chris

Re: [PATCH 0/2] list of gcc supported attribute name

[Luc Van Oostenryck]

On Mon, Jul 03, 2017 at 11:35:59AM -0700, Christopher Li wrote:
> This patch add full list of known gcc supported attributes.
> 
> So sparse should not give warning on the legit gcc attribute
> any more.
> 
> Test with full kernel allmodeconfig compile with the exact
> same output as before(master branch).
> 
> I want to include this two patches in the sparse release 0.5.1.
> 
> Git repository here:
> https://git.kernel.org/pub/scm/devel/sparse/sparse.git/log/?h=sparse-next-20170603

I think you meant -20170703 here, not -20170603.

On the mailing list, can you please send the patches inlined,
as, for review, attachements are kinda useless.

In patch 1, in the die() message, I think you meant:
	"created with different types"
               ^                    ^

In both patches, your signoff is supposed to contains your name.

Thanks,
-- Luc 

Re: [PATCH 0/2] list of gcc supported attribute name

[Christopher Li]

On Tue, Jul 4, 2017 at 1:05 AM, Luc Van Oostenryck
<luc.vanoostenryck@gmail.com> wrote:
> On Mon, Jul 03, 2017 at 11:35:59AM -0700, Christopher Li wrote:
>
> I think you meant -20170703 here, not -20170603.

Right.

>
> On the mailing list, can you please send the patches inlined,
> as, for review, attachements are kinda useless.

Will do.

>
> In patch 1, in the die() message, I think you meant:
>         "created with different types"
>                ^                    ^
>
> In both patches, your signoff is supposed to contains your name.

Thanks you very much for catching that.
I correct both of them. The branch has refreshed as

https://git.kernel.org/pub/scm/devel/sparse/sparse.git/log/?h=sparse-next-20170704

Chris