On 2013-07-11 17:24, Martin Storsjö wrote:
On Thu, 11 Jul 2013, Diego Biurrun wrote:
On 2013-07-11 16:23, Martin Storsjö wrote:
--- a/configure
+++ b/configure
@@ -3405,6 +3406,8 @@ unsigned int endian = 'B' << 24 | 'I' << 16 |
'G' << 8 | 'E';
  EOF
  od -t x1 $TMPO | grep -q '42 *49 *47 *45' && enable bigendian

+check_inline_asm inline_asm_labels '"1:\n"'
+
  if enabled alpha; then

      check_cflags -mieee

I guess this is more general than just AIX PPC, but AIX PPC is the
only affected platform AFAIU.  I'm somewhat undecided if we shouldn't
restrict this to PPC then.

Well, labels in inline assembly is something that is supported on
most/all platforms that gas/gcc supports, afaik. And if we have the
HAVE_INLINE_ASM_LABELS item, it would feel weird to have it set to 0 on
most platforms except PPC. So I'd rather keep the check generic.

Fine.

Would there be any point to putting this behind a "enabled inline_asm &&
..."? I don't think doing this check would hurt even if the compiler
doesn't support inline asm. (It doesn't hurt for MSVC.)

It does not, except for the wasted cycles.  Feel free to push whenever.

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to