flac dev - Feb 2017 - [PATCH] fix LLVM compile

Hi guys,

Please find attached a patch that fixes the OSX build with llvm
only __asm__ is supported but GCC support both so it's not an issue using
this one AFAIK.

lvqcl can probably confirm this.

Thanks !



-- 
Olivier Tristan
Research & Development
www.uvi.net
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.xiph.org/pipermail/flac-dev/attachments/20170227/631ac282/attachment.html>
-------------- next part --------------
diff --git a/src/libFLAC/cpu.c b/src/libFLAC/cpu.c
index 67a076d..bf0708c 100644
--- a/src/libFLAC/cpu.c
+++ b/src/libFLAC/cpu.c
@@ -81,7 +81,7 @@ cpu_xgetbv_x86(void)
 	return (uint32_t)_xgetbv(0);
 #elif defined __GNUC__
 	uint32_t lo, hi;
-	asm volatile (".byte 0x0f, 0x01, 0xd0" : "=a"(lo),
"=d"(hi) : "c" (0));
+	__asm__ volatile (".byte 0x0f, 0x01, 0xd0" : "=a"(lo),
"=d"(hi) : "c" (0));
 	return lo;
 #else
 	return 0;

Olivier Tristan wrote:
> Hi guys,
>
> Please find attached a patch that fixes the OSX build with llvm
> only __asm__ is supported but GCC support both so it's not an issue  
> using this one AFAIK.
>
> lvqcl can probably confirm this.
I can obly confirm that it works in GCC/MinGW, but that's expected.
Also, ffmpeg uses __asm__  for inline x86 assembly.

Olivier Tristan wrote:
> Please find attached a patch that fixes the OSX build with llvm
> only __asm__ is supported but GCC support both so it's not an issue
using
> this one AFAIK.
Applied. Thanks.

Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/