[vlc-devel] [PATCH] contrib/gcrypt: fix compilation with clang

[Rafaël Carré]

Hi,

Thanks for sending the patch, I think I need more details though to
understand what happens

On 02/13/14 20:22, Felix Paul Kühne wrote:
> ---
>  .../gcrypt/gcrypt-fix-compilation-with-clang.patch | 24 ++++++++++++++++++++++
>  contrib/src/gcrypt/rules.mak                       |  3 +++
>  2 files changed, 27 insertions(+)
>  create mode 100644 contrib/src/gcrypt/gcrypt-fix-compilation-with-clang.patch
> 
> diff --git a/contrib/src/gcrypt/gcrypt-fix-compilation-with-clang.patch b/contrib/src/gcrypt/gcrypt-fix-compilation-with-clang.patch
> new file mode 100644
> index 0000000..fd7497a
> --- /dev/null
> +++ b/contrib/src/gcrypt/gcrypt-fix-compilation-with-clang.patch
> @@ -0,0 +1,24 @@
> +diff -ru libgcrypt/mpi/amd64/mpih-mul1.S libgcrypt-fixed/mpi/amd64/mpih-mul1.S
> +--- libgcrypt/mpi/amd64/mpih-mul1.S	2013-03-15 20:25:38.000000000 +0100
> ++++ libgcrypt-fixed/mpi/amd64/mpih-mul1.S	2014-02-13 20:15:16.000000000 +0100
> +@@ -40,7 +40,7 @@
> + 
> + 
> + 	TEXT
> +-	ALIGN(5)
> ++	ALIGN(2)
> + 	.byte	0,0,0,0,0,0,0,0,0,0,0,0,0,0,0
> + 
> + 	GLOBL	C_SYMBOL_NAME(_gcry_mpih_mul_1)

What is the compilation error for this file?

> +diff -ru libgcrypt/mpi/amd64/mpih-rshift.S libgcrypt-fixed/mpi/amd64/mpih-rshift.S
> +--- libgcrypt/mpi/amd64/mpih-rshift.S	2013-03-15 20:25:38.000000000 +0100
> ++++ libgcrypt-fixed/mpi/amd64/mpih-rshift.S	2014-02-13 20:15:34.000000000 +0100
> +@@ -56,7 +56,7 @@
> + 	addq	$2, %rdx
> + 	jg	.Lendo
> + 
> +-	ALIGN(8)			/* minimal alignment for claimed speed */
> ++	ALIGN(2)			/* minimal alignment for claimed speed */
> + .Loop:	movq	-8(%rsi,%rdx,8), %mm6
> + 	movq	%mm6, %mm2
> + 	psllq	%mm0, %mm6

And this one?