Re: [PATCH 1/1] parser: add support for asm goto

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/18/2010 10:21 AM, Jiri Slaby wrote:
> Holy crap, in docco, there is nothing like that, but I checked gcc
> parser sources to be sure, and there indeed is:
>      asm type-qualifier[opt] ( asm-argument ) ;
>      asm type-qualifier[opt] goto ( asm-goto-argument ) ;
> So yes, asm volatile goto is valid. Would you take care of that?

It is as easy as:
diff --git a/parse.c b/parse.c
index caf10b9..1d20658 100644
--- a/parse.c
+++ b/parse.c
@@ -1915,7 +1915,8 @@ static struct token *parse_asm_statement(struct
token *token, struct statement *
        stmt->type = STMT_ASM;
        if (match_idents(token, &__volatile___ident, &__volatile_ident,
&volatile_ident, NULL)) {
                token = token->next;
-       } else if (match_idents(token, &goto_ident, NULL)) {
+       }
+       if (match_idents(token, &goto_ident, NULL)) {
                is_goto = 1;
                token = token->next;
        }

thanks,
-- 
js
suse labs
--
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


[Index of Archives]     [Newbies FAQ]     [LKML]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Trinity Fuzzer Tool]

  Powered by Linux