scanner: drop altmacro from dtddata.S
authorPekka Paalanen <pekka.paalanen@collabora.co.uk>
Wed, 18 Nov 2015 11:00:00 +0000 (13:00 +0200)
committerPekka Paalanen <pekka.paalanen@collabora.co.uk>
Thu, 19 Nov 2015 07:48:11 +0000 (09:48 +0200)
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@collabora.co.uk>
Tested-by: Víctor Jáquez <vjaquez@igalia.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
src/dtddata.S

index 53debf3..68e3435 100644 (file)
 
 /* 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