Skip to content

Added support for Asm labels#54

Merged
inducer merged 1 commit intoinducer:masterfrom
t11230:asm-labels
Aug 16, 2020
Merged

Added support for Asm labels#54
inducer merged 1 commit intoinducer:masterfrom
t11230:asm-labels

Conversation

@t11230
Copy link
Copy Markdown
Contributor

@t11230 t11230 commented Aug 15, 2020

  • Fixed superclass for ArrayDeclExt
  • Added some spaces for readability to _generate_type
  • Moved templated rules into _AsmAndAttributesMixin

@t11230
Copy link
Copy Markdown
Contributor Author

t11230 commented Aug 15, 2020

This should resolve #33

@inducer
Copy link
Copy Markdown
Owner

inducer commented Aug 15, 2020

This has some conflicts now. Could you resolve those?

@inducer
Copy link
Copy Markdown
Owner

inducer commented Aug 15, 2020

Thanks for working on this! Changes LGTM. Ready to go in once the conflicts are resolved.

- Fixed superclass for ArrayDeclExt
- Added some spaces for readability to _generate_type
- Moved templated rules into _AsmAndAttributesMixin
@inducer inducer merged commit 361679a into inducer:master Aug 16, 2020
@inducer
Copy link
Copy Markdown
Owner

inducer commented Aug 16, 2020

Thanks!

DCNick3 added a commit to DCNick3/pycparserext that referenced this pull request May 29, 2021
- Add support for declarations like `int (* __attribute__((stdcall)) fptr)(void)`
- Fix regression introduced in inducer#54, which caused function pointer information to be discarded in case any attributes or asm labels were present
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants