Skip to content

keyword driven parsing

Christopher Li linux-sparse 1 version Applied

Status from this message: Thanks, applied.

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 4 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/5Moving statement parsing into smaller functions.parse.c+139 -930 replies
2/5Free up some special bits in modifiers.parse.c parse.h symbol.c +2 more+133 -670 replies
2/5Introducing statement keywordsparse.c symbol.h+99 -380 replies
4/5Introduce top level parsing for asm parsing.parse.c symbol.h+8 -20 replies
5/5Introduce keyword driven attribute parsingparse.c symbol.h+262 -1930 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/5] keyword driven parsing

Here is an attempt to use keyword driven parsing.

The effect is that, we don't need to keep on adding keyword to ident-list any more.

It might be marginally faster because it does not do a series of testing for ident any more. I can't really tell because sparse is already fast.

It almost remove the usage match_idents(). The only place for using it now is parse_asm().

BTW, why asm is a statement not an expression? If it is an expression, it seems make sense to merge the volatile to the ctype modifiers.

Feel back is welcome.

Chris