Changeset 10995 in vbox

Timestamp:

Jul 30, 2008 3:49:16 PM (16 years ago)

Author:

vboxsync

Message:

iprt: Added ASMByteSwapU16/64 and unit test for them + U32.

Location:

trunk

Files:

• include/iprt/asm.h (modified) (2 diffs)
• src/VBox/Runtime/testcase/tstInlineAsm.cpp (modified) (2 diffs)

trunk/include/iprt/asm.h

@@ -5540 +5540 @@
 }
 
+/**
+ * Reverse the byte order of the given 16-bit integer.
+ *
+ * @returns Revert
+ * @param   u16     16-bit integer value.
+ */
+DECLINLINE(uint16_t) ASMByteSwapU16(uint16_t u16)
+{
+#if RT_INLINE_ASM_USES_INTRIN
+    u16 = _byteswap_ushort(u16);
+#elif RT_INLINE_ASM_GNU_STYLE
+    __asm__ ("rorw $8, %0" : "=r" (u16) : "0" (u16));
+#else
+    _asm
+    {
+        mov     ax, [u16]
+        ror     ax, 8
+        mov     [u16], ax
+    }
+#endif
+    return u16;
+}
 
 /**
  * Reverse the byte order of the given 32-bit integer.
- * @param  u32      Integer
+ *
+ * @returns Revert
+ * @param   u32     32-bit integer value.
  */
 DECLINLINE(uint32_t) ASMByteSwapU32(uint32_t u32)
@@ -5562 +5586 @@
 }
 
+
+/**
+ * Reverse the byte order of the given 64-bit integer.
+ *
+ * @returns Revert
+ * @param   u64     64-bit integer value.
+ */
+DECLINLINE(uint64_t) ASMByteSwapU64(uint64_t u64)
+{
+#if defined(RT_ARCH_AMD64) && RT_INLINE_ASM_USES_INTRIN
+    u64 = _byteswap_uint64(u64);
+#else /* !RT_ARCH_AMD64 (assume x86) */
+    u64 = (uint64_t)ASMByteSwapU32((uint32_t)u64) << 32
+        | (uint64_t)ASMByteSwapU32(u64 >> 32);
+#endif
+    return u64;
+}
+
+
 /** @} */
 

trunk/src/VBox/Runtime/testcase/tstInlineAsm.cpp

@@ -1071 +1071 @@
 }
 
-/*
- * Make this static. We don't want to have this located on the stack.
- */
+
+void tstASMByteSwap(void)
+{
+    RTPrintf("tstInlineASM: TESTING - ASMByteSwap*\n");
+
+    uint64_t u64In  = UINT64_C(0x0011223344556677);
+    uint64_t u64Out = ASMByteSwapU64(u64In);
+    CHECKVAL(u64In, UINT64_C(0x0011223344556677), "%#018RX64");
+    CHECKVAL(u64Out, UINT64_C(0x7766554433221100), "%#018RX64");
+    u64Out = ASMByteSwapU64(u64Out);
+    CHECKVAL(u64Out, u64In, "%#018RX64");
+    u64In  = UINT64_C(0x0123456789abcdef);
+    u64Out = ASMByteSwapU64(u64In);
+    CHECKVAL(u64In, UINT64_C(0x0123456789abcdef), "%#018RX64");
+    CHECKVAL(u64Out, UINT64_C(0xefcdab8967452301), "%#018RX64");
+    u64Out = ASMByteSwapU64(u64Out);
+    CHECKVAL(u64Out, u64In, "%#018RX64");
+    u64In  = 0;
+    u64Out = ASMByteSwapU64(u64In);
+    CHECKVAL(u64Out, u64In, "%#018RX64");
+    u64In  = ~(uint64_t)0;
+    u64Out = ASMByteSwapU64(u64In);
+    CHECKVAL(u64Out, u64In, "%#018RX64");
+
+    uint32_t u32In  = UINT32_C(0x00112233);
+    uint32_t u32Out = ASMByteSwapU32(u32In);
+    CHECKVAL(u32In, UINT32_C(0x00112233), "%#010RX32");
+    CHECKVAL(u32Out, UINT32_C(0x33221100), "%#010RX32");
+    u32Out = ASMByteSwapU32(u32Out);
+    CHECKVAL(u32Out, u32In, "%#010RX32");
+    u32In  = UINT32_C(0x12345678);
+    u32Out = ASMByteSwapU32(u32In);
+    CHECKVAL(u32In, UINT32_C(0x12345678), "%#010RX32");
+    CHECKVAL(u32Out, UINT32_C(0x78563412), "%#010RX32");
+    u32Out = ASMByteSwapU32(u32Out);
+    CHECKVAL(u32Out, u32In, "%#010RX32");
+    u32In  = 0;
+    u32Out = ASMByteSwapU32(u32In);
+    CHECKVAL(u32Out, u32In, "%#010RX32");
+    u32In  = ~(uint32_t)0;
+    u32Out = ASMByteSwapU32(u32In);
+    CHECKVAL(u32Out, u32In, "%#010RX32");
+
+    uint16_t u16In  = UINT16_C(0x0011);
+    uint16_t u16Out = ASMByteSwapU16(u16In);
+    CHECKVAL(u16In, UINT16_C(0x0011), "%#06RX16");
+    CHECKVAL(u16Out, UINT16_C(0x1100), "%#06RX16");
+    u16Out = ASMByteSwapU16(u16Out);
+    CHECKVAL(u16Out, u16In, "%#06RX16");
+    u16In  = UINT16_C(0x1234);
+    u16Out = ASMByteSwapU16(u16In);
+    CHECKVAL(u16In, UINT16_C(0x1234), "%#06RX16");
+    CHECKVAL(u16Out, UINT16_C(0x3412), "%#06RX16");
+    u16Out = ASMByteSwapU16(u16Out);
+    CHECKVAL(u16Out, u16In, "%#06RX16");
+    u16In  = 0;
+    u16Out = ASMByteSwapU16(u16In);
+    CHECKVAL(u16Out, u16In, "%#06RX16");
+    u16In  = ~(uint16_t)0;
+    u16Out = ASMByteSwapU16(u16In);
+    CHECKVAL(u16Out, u16In, "%#06RX16");
+}
+
+
 void tstASMBench(void)
 {
+    /*
+     * Make this static. We don't want to have this located on the stack.
+     */
     static uint8_t  volatile s_u8;
     static int8_t   volatile s_i8;
@@ -1175 +1239 @@
     tstASMMemFill32();
     tstASMMath();
+    tstASMByteSwap();
 
     tstASMBench();