[PATCH wayland 2/2] scanner: drop altmacro from dtddata.S
Pekka Paalanen
ppaalanen at gmail.com
Wed Nov 18 03:07:37 PST 2015
From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
Stop using .altmacro in dtddata.S, because clang does not yet implement
it. Turns out that we do not actually seem to need it, and we can modify
the syntax to work without it.
Moving the double quotes from the binfile line to the .incbin line is
required to avoid the assembler error "missing string". Instead of & we
now use \() to mark the end of macro argument name.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92988
Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
---
src/dtddata.S | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/src/dtddata.S b/src/dtddata.S
index 53debf3..68e3435 100644
--- a/src/dtddata.S
+++ b/src/dtddata.S
@@ -22,19 +22,18 @@
/* from: http://www.linuxjournal.com/content/embedding-file-executable-aka-hello-world-version-5967#comment-348129 */
-.altmacro
.macro binfile name file
.p2align 2
- .globl \name&_begin
-\name&_begin:
- .incbin \file
-\name&_end:
+ .globl \name\()_begin
+\name\()_begin:
+ .incbin "\file"
+\name\()_end:
.byte 0
.p2align 2
- .globl \name&_len
-\name&_len:
- .int (\name&_end - \name&_begin)
+ .globl \name\()_len
+\name\()_len:
+ .int (\name\()_end - \name\()_begin)
.endm
.section .rodata
-binfile DTD_DATA "src/wayland.dtd.embed"
+binfile DTD_DATA src/wayland.dtd.embed
--
2.4.10
More information about the wayland-devel
mailing list