Skip to content

Commit

Permalink
app-arch/lzop: add patch from OE for building on x32 #575450
Browse files Browse the repository at this point in the history
  • Loading branch information
vapier committed Aug 10, 2016
1 parent 2cba5f5 commit 2583f48
Show file tree
Hide file tree
Showing 2 changed files with 42 additions and 0 deletions.
38 changes: 38 additions & 0 deletions app-arch/lzop/files/lzop-1.03-x32.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
https://bugs.gentoo.org/575450

Fix get of FLAGS register in x32 ABI,

x32 ABI requires to have 64-bit variable to store FLAGS register
instead of size_t that is 32-bit variable, this fix operand
type mismatch when try to pop previous pushf value.

Upstream-status: Submitted

Signed-off-by: Aníbal Limón <[email protected]>

Index: lzop-1.03/src/miniacc.h
===================================================================
--- lzop-1.03.orig/src/miniacc.h
+++ lzop-1.03/src/miniacc.h
@@ -754,6 +754,9 @@
#elif defined(__amd64__) || defined(__x86_64__) || defined(_M_AMD64)
# define ACC_ARCH_AMD64 1
# define ACC_INFO_ARCH "amd64"
+# if defined(__ILP32__)
+# define ACC_ARCH_AMD64_X32 1
+# endif
#elif defined(__thumb__) || (defined(_M_ARM) && defined(_M_THUMB))
# define ACC_ARCH_ARM 1
# define ACC_ARCH_ARM_THUMB 1
@@ -6787,7 +6790,11 @@ ACCLIB_PUBLIC_NOINLINE(void, acc_debug_n
ACCLIB_PUBLIC_NOINLINE(int, acc_debug_align_check_query) (void)
{
#if (ACC_ARCH_AMD64 || ACC_ARCH_I386) && (ACC_ASM_SYNTAX_GNUC)
+# if defined(ACC_ARCH_AMD64_X32)
+ unsigned long long r;
+# else
size_t r;
+# endif
__asm__ __volatile__("pushf\n pop %0\n" : "=a" (r) : : __ACC_ASM_CLOBBER);
return (int)(r >> 18) & 1;
#elif (ACC_ARCH_I386) && (ACC_ASM_SYNTAX_MSC)
4 changes: 4 additions & 0 deletions app-arch/lzop/lzop-1.03.ebuild
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ IUSE=""
RDEPEND=">=dev-libs/lzo-2"
DEPEND="${RDEPEND}"

PATCHES=(
"${FILESDIR}"/${P}-x32.patch #575450
)

src_test() {
einfo "compressing config.status to test"
src/lzop config.status || die 'compression failed'
Expand Down

0 comments on commit 2583f48

Please sign in to comment.