On 2014-08-03 12:23:35 +0300, Martin Storsjö wrote:
> On Sun, 3 Aug 2014, Martin Storsjö wrote:
> 
> >On Thu, 24 Jul 2014, Janne Grunau wrote:
> >
> >>---
> >>libavutil/aarch64/asm.S | 9 ++++++++-
> >>1 file changed, 8 insertions(+), 1 deletion(-)
> >>
> >>diff --git a/libavutil/aarch64/asm.S b/libavutil/aarch64/asm.S
> >>index 850f16b..f765e59 100644
> >>--- a/libavutil/aarch64/asm.S
> >>+++ b/libavutil/aarch64/asm.S
> >>@@ -26,6 +26,12 @@
> >>#   define ELF  #
> >>#endif
> >>
> >>+#ifdef __MACH__
> >>+#   define MACH
> >>+#else
> >>+#   define MACH #
> >>+#endif
> >>+
> >>#if HAVE_AS_FUNC
> >>#   define FUNC
> >>#else
> >>@@ -57,7 +63,8 @@ FUNC    .func   \name
> >>ELF     .size   \name, . - \name
> >>        .purgem endconst
> >>    .endm
> >>-        .section        .rodata
> >>+ELF     .section        .rodata
> >>+MACH    .const_data
> >>        .align          \align
> >>\name:
> >>.endm
> >>-- 
> >>2.0.1
> >
> >Ok
> 
> Actually - IIRC .section .rodata also works with binutils for
> windows as well (we do that unconditionally in the 32 bit arm
> asm.S), so if there's ever support for aarch64 on windows, we might
> not want to have that under ELF, but under "non-mach".

yes, I was unsure about this one. It would also create problems for 
gas-preprocessor armasm. I'll redo it.

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to