Changeset 103929 in vbox

Timestamp:

Mar 19, 2024 11:43:54 PM (13 months ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

162303

Message:

iprt/x86.h: Added X86_OP_VEX3_BYTE1_MAKE(). bugref:9898

File:

• trunk/include/iprt/x86.h (modified) (2 diffs)

trunk/include/iprt/x86.h ¶

@@ -5215 +5215 @@
 #define X86_OP_VEX3_BYTE1_X             RT_BIT(6)
 #define X86_OP_VEX3_BYTE1_R             RT_BIT(7)
+#define X86_OP_VEX3_BYTE1_MAKE(a_idxMap, a_B, a_X, a_R) \
+     (  (uint8_t)(a_idxMap) \
+      | ((a_B) ? 0 : X86_OP_VEX3_BYTE1_B) \
+      | ((a_X) ? 0 : X86_OP_VEX3_BYTE1_X) \
+      | ((a_R) ? 0 : X86_OP_VEX3_BYTE1_R))
 
 #define X86_OP_VEX3_BYTE2_P_MASK        0x3
@@ -5227 +5232 @@
 #define X86_OP_VEX3_BYTE2_W             RT_BIT(7)
 
-#define X86_OP_VEX3_BYTE2_MAKE(a_f64BitOpSz, a_iSrcReg, a_f256BitAvx, a_fPrf) \
-    (  ((a_f64BitOpSz) ? X86_OP_VEX3_BYTE2_W : 0) \
+/** @todo r=bird: Is the '& UINT8_C(0xf)' bit needed? You mask it again after
+ *        shifting. */
+#define X86_OP_VEX3_BYTE2_MAKE(a_f64BitOpSize, a_iSrcReg, a_f256BitAvx, a_fPrf) \
+    (  ((a_f64BitOpSize) ? X86_OP_VEX3_BYTE2_W : 0) \
      | ((~((uint8_t)(a_iSrcReg) & UINT8_C(0xf)) << X86_OP_VEX3_BYTE2_VVVV_SHIFT) & X86_OP_VEX3_BYTE2_VVVV_MASK) \
      | ((a_f256BitAvx) ? X86_OP_VEX3_BYTE2_L : 0) \
      | ((a_fPrf) & X86_OP_VEX3_BYTE2_P_MASK))
 
-#define X86_OP_VEX3_BYTE2_MAKE_NO_VVVV(a_f64BitOpSz, a_f256BitAvx, a_fPrf) \
-    (  ((a_f64BitOpSz) ? X86_OP_VEX3_BYTE2_W : 0) \
+#define X86_OP_VEX3_BYTE2_MAKE_NO_VVVV(a_f64BitOpSize, a_f256BitAvx, a_fPrf) \
+    (  ((a_f64BitOpSize) ? X86_OP_VEX3_BYTE2_W : 0) \
      | (X86_OP_VEX3_BYTE2_VVVV_NONE << X86_OP_VEX3_BYTE2_VVVV_SHIFT) \
      | ((a_f256BitAvx) ? X86_OP_VEX3_BYTE2_L : 0) \