All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] sparse: ignore __assume_aligned__ attribute
@ 2016-09-21 14:44 Lance Richardson
  2016-09-28 18:17 ` Christopher Li
  0 siblings, 1 reply; 2+ messages in thread
From: Lance Richardson @ 2016-09-21 14:44 UTC (permalink / raw)
  To: linux-sparse

The __assume_aligned__ attribute can be safely ignored, add it
to the list of ignored attributes.

Signed-off-by: Lance Richardson <lrichard@redhat.com>
---
 parse.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/parse.c b/parse.c
index 98a31dc..2a1b57e 100644
--- a/parse.c
+++ b/parse.c
@@ -526,6 +526,8 @@ const char *ignored_attributes[] = {
 	"__always_inline__",
 	"artificial",
 	"__artificial__",
+	"assume_aligned",
+	"__assume_aligned__",
 	"bounded",
 	"__bounded__",
 	"cdecl",
-- 
2.5.5


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] sparse: ignore __assume_aligned__ attribute
  2016-09-21 14:44 [PATCH] sparse: ignore __assume_aligned__ attribute Lance Richardson
@ 2016-09-28 18:17 ` Christopher Li
  0 siblings, 0 replies; 2+ messages in thread
From: Christopher Li @ 2016-09-28 18:17 UTC (permalink / raw)
  To: Lance Richardson; +Cc: Linux-Sparse

On Wed, Sep 21, 2016 at 7:44 AM, Lance Richardson <lrichard@redhat.com> wrote:
> The __assume_aligned__ attribute can be safely ignored, add it
> to the list of ignored attributes.

Thanks for the patch. Sorry for the delay. The change looks fine.

Can you add some test case of the  __assume_aligned__ attribute?
I want to see how it was used in your source code which cause the problem
for sparse.

We need some basic test case just in case we parse the attributes in the
future.

Thanks

Chris


>
> Signed-off-by: Lance Richardson <lrichard@redhat.com>
> ---
>  parse.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/parse.c b/parse.c
> index 98a31dc..2a1b57e 100644
> --- a/parse.c
> +++ b/parse.c
> @@ -526,6 +526,8 @@ const char *ignored_attributes[] = {
>         "__always_inline__",
>         "artificial",
>         "__artificial__",
> +       "assume_aligned",
> +       "__assume_aligned__",
>         "bounded",
>         "__bounded__",
>         "cdecl",
> --
> 2.5.5
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-09-28 18:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-21 14:44 [PATCH] sparse: ignore __assume_aligned__ attribute Lance Richardson
2016-09-28 18:17 ` Christopher Li

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.