linux-kernel - Re: [llvmlinux] [PATCH] [RFC] [X86] Fix a compilation issue with clang.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <1377803585-5913-1-git-send-email-dl9pf@gmx.de>
Date:	Thu, 29 Aug 2013 21:13:03 +0200
From:	dl9pf@....de
To:	dl9pf@....de
Cc:	pageexec@...email.hu, llvmlinux@...ts.linuxfoundation.org,
	behanw@...verseincode.com, hpa@...or.com,
	linux-kernel@...r.kernel.org
Subject: Re: [llvmlinux] [PATCH] [RFC] [X86] Fix a compilation issue with clang.

These to patches fix a compilation issues with clang.

[PATCH 1/2] Extend definitions of __ASM_* with a raw format.

Extends the helper macros __ASM_*  (e.g. __ASM_DX) with a format that 
does not add whitespace. Both gcc and clang choke in that case.

[PATCH 2/2] [X86] Fix a compilation issue with clang.

This is the actual issue here. In the 64bit case, clang just sees
edx (a 32bit register name) and does not use more than 32bit.
As pointed out in the comment in uaccess.h, gcc is happy with edx as the
starting point (and does not care about the size).
This fix pleases both worlds as we just use edx on 32 and rdx on 64.

Please apply.

CC: pageexec@...email.hu
CC: llvmlinux@...ts.linuxfoundation.org
CC: behanw@...verseincode.com
CC: hpa@...or.com
CC: linux-kernel@...r.kernel.org
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/