[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[XEN PATCH] xen: Drop logic for old clang versions.
- To: xen-devel@xxxxxxxxxxxxxxxxxxxx
- From: nicola.vetrini@xxxxxxxxx
- Date: Sun, 10 Aug 2025 15:03:53 +0200
- Arc-authentication-results: i=1; bugseng.com; arc=none smtp.remote-ip=46.228.253.214
- Arc-message-signature: i=1; d=bugseng.com; s=openarc; a=rsa-sha256; c=relaxed/relaxed; t=1754831116; h=From:To:Cc:Subject:Date:Message-ID:X-Mailer:MIME-Version: Content-Transfer-Encoding; bh=Wmwa6eEi2HUkBCkPAQOZ2YgwGrySmusYzIvJZGl4xUw=; b=tS8oeWr19VM4yJtAZKh+tUG0Zfg45inl5Vw6x5GteVziHGXPAPKDY8Okk2wF+fjO5RI0 rM/nQXocpjyOiOMpbyeZc4f5XjkpKSUIaSIR0wzBFrqRMLJbflOqDy6hsstazC4Q09Bz2 UnEtUmqON5kd/T9L3I9xYfX3pFmLpsdA2X7sd7gxnMxOu9ljLI33clk9L30v+1A++vxl8 9mvyVwn+9me4EMkx3nfmCZn8sBsM10IfMNBoARJx/SciJXjefKp3muHXgGpWuEYOXjSqy bNuPA9GRpc9S92xeI30gzJ/gAcx79P7onqMeUsIO+lYPnjPfwJTnkruuSK+JYrDrmcghs +7IYCxW/v2KnpaWigr9S4tBzXl6TmyNQpPyqZcOzQJhkC2CmNQ/bNFuMZZ/4gb2XdkMSz H7CTmdIvTHYNZ4jTYzBK4NB8IND3JVL60lldcjDBoOYlWRri3+ETvTQjQXyx3vZhZd/n3 YBaJoOE0eU47fjYswIbKbIe70Yrot3fKEXlEJQBOw3dp8FSFIBXTKGbgGjLAc5WqeCe/d 6LNrYnJDV4dBuZ/g+jY1WQFGBg4hxkD6HRCDvCVqe0yz1jV7sBVOZnHIa0+CWC1sLG8Pc 99DZ6VHFhm4jdEfS6xHBkCHP8ftG86vpkZNdyPPh+TWTrOfM+z3xqfXgw+5d4FM=
- Arc-seal: i=1; d=bugseng.com; s=openarc; a=rsa-sha256; cv=none; t=1754831116; b=fpKYXOsawbT/B0gieMLHTKuUgi3fgyM1C8YDH7ksLEPdgalPDTveiYFF474U3Y6Q3BMy 1Z82uxdVNPVQwVFQRYpU0dMdiw9/Ylz6W5US+TjW7B2RFyY7habJHyL4AwHvVnOI+Awfg o0X28Mv51m8ZQ9bR6lXRjRLW/gGW8ZmU00H52MdFw7h2DXhQDGOvkHLjf1W20gRTqap2i cWI8D0PZNXvpOGaXYnmztLIipm6wQuB+D3TIpfKPsPDVzhMYw9QWC6yEifk54hx+/bbKq w7pMFPSQiFt2Qy//ixFwC8+YGI60QjCkfFkrgWn4Pw2KIIFJvzmd+1MkhUIMnM8Gk7WU4 us0g0i5BoPAGDjmVItW+rMfZPkRyDPhFm1mxIyaO1Kr/NO6sXJFIyndm6SxKG6bvyg1pj ezrw4yRwxRgMP++dyeXYcHfLpfaofPuxM3Ndph4EXLYDvL05Vd1Bu/UjxNRCKN+PNQU9H MM6RkficDc1Ry8bSrt4/P1m8khtCqKXpQCWrbIKt1LwyMvFK+dn+H2yxwEqqq+ZR3noxQ jl5Z/8jrXgCGKjNSG79zFG9pVFbE0JaG0dC4+qDzRFt07aWjQkhYKS0+1rYerDkyPgS1F kE/xF7GZgp5zL9mcTJOvwuBcY7vcmw0eHXHdV/wS7ivrUKWukkYuA6yvMbr/s4o=
- Authentication-results: bugseng.com; arc=none smtp.remote-ip=46.228.253.214
- Cc: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Julien Grall <julien@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Nicola Vetrini <nicola.vetrini@xxxxxxxxx>
- Delivery-date: Sun, 10 Aug 2025 13:06:41 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
From: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
The enforced toolchain baseline for clang is version 11,
therefore this logic is effectively dead code.
No functional change.
Signed-off-by: Nicola Vetrini <nicola.vetrini@xxxxxxxxx>
---
Mentioned in https://gitlab.com/xen-project/xen/-/issues/201
---
xen/common/coverage/llvm.c | 4 ----
xen/include/xen/self-tests.h | 9 +--------
2 files changed, 1 insertion(+), 12 deletions(-)
diff --git a/xen/common/coverage/llvm.c b/xen/common/coverage/llvm.c
index 50d7a3c5d301..517b2aa8c202 100644
--- a/xen/common/coverage/llvm.c
+++ b/xen/common/coverage/llvm.c
@@ -44,12 +44,8 @@
((uint64_t)'f' << 16) | ((uint64_t)'R' << 8) | ((uint64_t)129)
#endif
-#if __clang_major__ >= 4 || (__clang_major__ == 3 && __clang_minor__ >= 9)
#define LLVM_PROFILE_VERSION 4
#define LLVM_PROFILE_NUM_KINDS 2
-#else
-#error "clang version not supported with coverage"
-#endif
struct llvm_profile_data {
uint64_t name_ref;
diff --git a/xen/include/xen/self-tests.h b/xen/include/xen/self-tests.h
index bd8a4867aa40..c57cceb3b962 100644
--- a/xen/include/xen/self-tests.h
+++ b/xen/include/xen/self-tests.h
@@ -10,16 +10,12 @@
#include <xen/lib.h>
/*
- * Check that fn(val) can be calcuated by the compiler, and that it gives the
+ * Check that fn(val) can be calculated by the compiler, and that it gives the
* expected answer.
*
- * Clang < 8 can't fold constants through static inlines, causing this to
- * fail. Simply skip it for incredibly old compilers.
- *
* N.B. fn is intentionally not bracketed to allow us to test function-like
* macros too.
*/
-#if !defined(CONFIG_CC_IS_CLANG) || CONFIG_CLANG_VERSION >= 80000
#define COMPILE_CHECK(fn, val, res) \
do { \
typeof(fn(val)) real = fn(val); \
@@ -29,9 +25,6 @@
else if ( real != (res) ) \
BUILD_ERROR("Compile time check '" STR(fn(val) == res) "'
failed"); \
} while ( 0 )
-#else
-#define COMPILE_CHECK(fn, val, res)
-#endif
/*
* Check that Xen's runtime logic for fn(val) gives the expected answer. This
--
2.43.0
|