https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87291
David Malcolm <dmalcolm at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #48684|0 |1
is obsolete| |
--- Comment #27 from David Malcolm <dmalcolm at gcc dot gnu.org> ---
Created attachment 48694
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=48694&action=edit
v3 of the work-in-progress patch
Here's an updated version of the patch; also uploaded to:
https://dmalcolm.fedorapeople.org/gcc/2020-06-06/0001-FIXME-WIP-on-extended-asm-support-v3.patch
In particular, this adds a new:
gcc_jit_context_add_top_level_asm
entrypoint. On implementing this, it hooks into the symbol table like the C
frontend does, so it looks like it will respect ordering as much as the C
frontend does.
How does this look? The top-level asm code is barely tested though; I'd
appreciate a way to verify it from the automated test case.
Changes in v3:
* added gcc_jit_context_add_top_level_asm
* drop redundant gcc_jit_extended_asm_add_goto_label
* added gcc_jit_extended_asm_as_object
* started adding documentation
* added comments to libgccjit.h
* consolidated test-asm.c, grouping create/verify pairs
* add test coverage for "volatile"
* initial implementation of make_debug_string
* added some error-checking
