Merge #333985: x265: fix build on x86_64-darwin

...into staging-next
This commit is contained in:
Vladimír Čunát 2024-08-12 08:46:11 +02:00
commit 99f578eb76
No known key found for this signature in database
GPG Key ID: E747DF1F9575A3AA
2 changed files with 35 additions and 0 deletions

View File

@ -0,0 +1,29 @@
From 5ad351f7d271d0be0611797542c831898b2f531c Mon Sep 17 00:00:00 2001
From: Sergei Trofimovich <slyich@gmail.com>
Date: Sun, 11 Aug 2024 22:09:34 +0100
Subject: [PATCH] source/test/testharness.h: don't redefine `__rdtsc()` builtin
On darwin clang-16 provides `__rdtsc()` builtin. As a result the build
fails in `nixpkgs` as:
source/test/testharness.h:78:24: error: static declaration of '__rdtsc' follows non-static declaration
static inline uint32_t __rdtsc(void)
^
x265_3.6/source/test/testharness.h:78:24: note: '__rdtsc' is a builtin with type 2
The change avoid redefinition on targets that define `__rdtsc()` builtin.
---
source/test/testharness.h | 2 ++
1 file changed, 2 insertions(+)
--- a/test/testharness.h
+++ b/test/testharness.h
@@ -69,6 +69,8 @@ protected:
#include <intrin.h>
#elif HAVE_RDTSC
#include <intrin.h>
+#elif defined(__has_builtin) && __has_builtin(__rdtsc)
+/* compiler-provided builtin */
#elif (!defined(__APPLE__) && (defined (__GNUC__) && (defined(__x86_64__) || defined(__i386__))))
#include <x86intrin.h>
#elif ( !defined(__APPLE__) && defined (__GNUC__) && defined(__ARM_NEON__))

View File

@ -41,6 +41,12 @@ stdenv.mkDerivation rec {
hash = "sha256-ZjUx80HFOJ9GDXMOYuEKT8yjQoyiyhCWk4Z7xf4uKAc=";
};
# TODO: apply patch unconditionally in staging. It's conditional to
# save rebuild on staging-next.
patches = lib.optionals (stdenv.isDarwin && stdenv.isx86_64) [
./darwin-__rdtsc.patch
];
sourceRoot = "x265_${version}/source";
postPatch = ''