qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[RFC PATCH v2 4/5] tests/tcg/ppc64le: emit bcdsub with .long when needed


From: matheus . ferst
Subject: [RFC PATCH v2 4/5] tests/tcg/ppc64le: emit bcdsub with .long when needed
Date: Thu, 3 Mar 2022 14:20:40 -0300

From: Matheus Ferst <matheus.ferst@eldorado.org.br>

Based on GCC docs[1], we use the '-mpower8-vector' flag at config-time
to detect the toolchain support to the bcdsub instruction. LLVM/Clang
supports this flag since version 3.6[2], but the instruction and related
builtins were only added in LLVM 14[3]. In the absence of other means to
detect this support at config-time, we resort to __has_builtin to
identify the presence of __builtin_bcdsub at compile-time. If the
builtin is not available, the instruction is emitted with a ".long".

[1] 
https://gcc.gnu.org/onlinedocs/gcc-8.3.0/gcc/PowerPC-AltiVec_002fVSX-Built-in-Functions.html
[2] 
https://github.com/llvm/llvm-project/commit/59eb767e11d4ffefb5f55409524e5c8416b2b0db
[3] 
https://github.com/llvm/llvm-project/commit/c933c2eb334660c131f4afc9d194fafb0cec0423

Signed-off-by: Matheus Ferst <matheus.ferst@eldorado.org.br>
---
 tests/tcg/ppc64le/bcdsub.c | 55 +++++++++++++++++++++-----------------
 1 file changed, 30 insertions(+), 25 deletions(-)

diff --git a/tests/tcg/ppc64le/bcdsub.c b/tests/tcg/ppc64le/bcdsub.c
index c9ca5357cb..445d50f07d 100644
--- a/tests/tcg/ppc64le/bcdsub.c
+++ b/tests/tcg/ppc64le/bcdsub.c
@@ -9,32 +9,37 @@
 #define CRF_SO  (1 << 0)
 #define UNDEF   0
 
-#define BCDSUB(AH, AL, BH, BL, PS)                          \
-    asm ("mtvsrd 32, %3\n\t"                                \
-         "mtvsrd 33, %4\n\t"                                \
-         "xxmrghd 32, 32, 33\n\t"                           \
-         "mtvsrd 33, %5\n\t"                                \
-         "mtvsrd 34, %6\n\t"                                \
-         "xxmrghd 33, 33, 34\n\t"                           \
-         "bcdsub. 0, 0, 1, %7\n\t"                          \
-         "mfocrf %0, 0b10\n\t"                              \
-         "mfvsrd %1, 32\n\t"                                \
-         "xxswapd 32, 32\n\t"                               \
-         "mfvsrd %2, 32\n\t"                                \
-         : "=r" (cr), "=r" (th), "=r" (tl)                  \
-         : "r" (AH), "r" (AL), "r" (BH), "r" (BL), "i" (PS) \
-         : "v0", "v1", "v2");
+#if defined(__has_builtin) && !__has_builtin(__builtin_bcdsub)
+#define BCDSUB(T, A, B, PS) \
+    ".long 4 << 26 | (" #T ") << 21 | (" #A ") << 16 | (" #B ") << 11"  \
+    " | 1 << 10 | (" #PS ") << 9 | 65\n\t"
+#else
+#define BCDSUB(T, A, B, PS) "bcdsub. " #T ", " #A ", " #B ", " #PS "\n\t"
+#endif
 
-#define TEST(AH, AL, BH, BL, PS, TH, TL, CR6)   \
-    do {                                        \
-        int cr = 0;                             \
-        uint64_t th, tl;                        \
-        BCDSUB(AH, AL, BH, BL, PS);             \
-        if (TH || TL) {                         \
-            assert(tl == TL);                   \
-            assert(th == TH);                   \
-        }                                       \
-        assert((cr >> 4) == CR6);               \
+#define TEST(AH, AL, BH, BL, PS, TH, TL, CR6)                   \
+    do {                                                        \
+        int cr = 0;                                             \
+        uint64_t th, tl;                                        \
+        asm ("mtvsrd 32, %3\n\t"                                \
+             "mtvsrd 33, %4\n\t"                                \
+             "xxmrghd 32, 32, 33\n\t"                           \
+             "mtvsrd 33, %5\n\t"                                \
+             "mtvsrd 34, %6\n\t"                                \
+             "xxmrghd 33, 33, 34\n\t"                           \
+             BCDSUB(0, 0, 1, PS)                                \
+             "mfocrf %0, 0b10\n\t"                              \
+             "mfvsrd %1, 32\n\t"                                \
+             "xxswapd 32, 32\n\t"                               \
+             "mfvsrd %2, 32\n\t"                                \
+             : "=r" (cr), "=r" (th), "=r" (tl)                  \
+             : "r" (AH), "r" (AL), "r" (BH), "r" (BL)           \
+             : "v0", "v1", "v2");                               \
+        if (TH || TL) {                                         \
+            assert(tl == TL);                                   \
+            assert(th == TH);                                   \
+        }                                                       \
+        assert((cr >> 4) == CR6);                               \
     } while (0)
 
 
-- 
2.25.1




reply via email to

[Prev in Thread] Current Thread [Next in Thread]