Changeset 98134 in vbox for trunk/src/VBox/VMM/VMMAll

Timestamp:

Jan 19, 2023 11:46:27 AM (2 years ago)

Author:

vboxsync

Message:

VMM/IEM: Implementation of roundxx SSE instructions. [clang build fixes] bugref:9898

File:

• trunk/src/VBox/VMM/VMMAll/IEMAllAImplC.cpp (modified) (13 diffs)

trunk/src/VBox/VMM/VMMAll/IEMAllAImplC.cpp

@@ -14460 +14460 @@
     }
 
+#ifdef IEM_WITHOUT_ASSEMBLY
 
 /**
@@ -14583 +14584 @@
 }
 
+#endif /* IEM_WITHOUT_ASSEMBLY */
+
 
 /**
@@ -14645 +14648 @@
 }
 
+#ifdef IEM_WITHOUT_ASSEMBLY
 
 /**
@@ -14658 +14662 @@
 DECLINLINE(bool) iemSseBinaryValIsNaNR32(PRTFLOAT32U pr32Res, PCRTFLOAT32U pr32Val1, PCRTFLOAT32U pr32Val2, uint32_t *pfMxcsr)
 {
-    uint8_t cQNan = RTFLOAT32U_IS_QUIET_NAN(pr32Val1) + RTFLOAT32U_IS_QUIET_NAN(pr32Val2);
-    uint8_t cSNan = RTFLOAT32U_IS_SIGNALLING_NAN(pr32Val1) + RTFLOAT32U_IS_SIGNALLING_NAN(pr32Val2);
+    uint8_t const cQNan = RTFLOAT32U_IS_QUIET_NAN(pr32Val1)      + RTFLOAT32U_IS_QUIET_NAN(pr32Val2);
+    uint8_t const cSNan = RTFLOAT32U_IS_SIGNALLING_NAN(pr32Val1) + RTFLOAT32U_IS_SIGNALLING_NAN(pr32Val2);
     if (cSNan + cQNan == 2)
     {
@@ -14668 +14672 @@
         return true;
     }
-    else if (cSNan)
+    if (cSNan)
     {
         /* One operand is an SNan and placed into the result, converting it to a QNan. */
@@ -14676 +14680 @@
         return true;
     }
-    else if (cQNan)
+    if (cQNan)
     {
         /* The QNan operand is placed into the result. */
@@ -14700 +14704 @@
 DECLINLINE(bool) iemSseBinaryValIsNaNR64(PRTFLOAT64U pr64Res, PCRTFLOAT64U pr64Val1, PCRTFLOAT64U pr64Val2, uint32_t *pfMxcsr)
 {
-    uint8_t cQNan = RTFLOAT64U_IS_QUIET_NAN(pr64Val1) + RTFLOAT64U_IS_QUIET_NAN(pr64Val2);
-    uint8_t cSNan = RTFLOAT64U_IS_SIGNALLING_NAN(pr64Val1) + RTFLOAT64U_IS_SIGNALLING_NAN(pr64Val2);
+    uint8_t const cQNan = RTFLOAT64U_IS_QUIET_NAN(pr64Val1)      + RTFLOAT64U_IS_QUIET_NAN(pr64Val2);
+    uint8_t const cSNan = RTFLOAT64U_IS_SIGNALLING_NAN(pr64Val1) + RTFLOAT64U_IS_SIGNALLING_NAN(pr64Val2);
     if (cSNan + cQNan == 2)
     {
@@ -14710 +14714 @@
         return true;
     }
-    else if (cSNan)
+    if (cSNan)
     {
         /* One operand is an SNan and placed into the result, converting it to a QNan. */
@@ -14718 +14722 @@
         return true;
     }
-    else if (cQNan)
+    if (cQNan)
     {
         /* The QNan operand is placed into the result. */
@@ -14749 +14753 @@
         return true;
     }
-    else if (RTFLOAT32U_IS_QUIET_NAN(pr32Val))
+    if (RTFLOAT32U_IS_QUIET_NAN(pr32Val))
     {
         /* The QNan operand is placed into the result. */
@@ -14779 +14783 @@
         return true;
     }
-    else if (RTFLOAT64U_IS_QUIET_NAN(pr64Val))
+    if (RTFLOAT64U_IS_QUIET_NAN(pr64Val))
     {
         /* The QNan operand is placed into the result. */
@@ -14789 +14793 @@
 }
 
+#endif /* IEM_WITHOUT_ASSEMBLY */
 
 /**
@@ -17057 +17062 @@
 
         RTFLOAT64U r64Src1, r64Src2;
-        uint32_t fDe  = iemSsePrepareValueR64(&r64Src1, *pfMxcsr, pr64Src1);
-                 fDe |= iemSsePrepareValueR64(&r64Src2, *pfMxcsr, pr64Src2);
+        uint32_t fDe  = iemSsePrepareValueR64(&r64Src1, *pfMxcsr, pr64Src1)
+                      | iemSsePrepareValueR64(&r64Src2, *pfMxcsr, pr64Src2);
 
         *pfMxcsr |= fDe;