Changeset 59747 in vbox for trunk/src/VBox/Devices/BiosCommonCode/MakeAlternativeSource.cpp

Timestamp:

Feb 19, 2016 11:18:18 PM (9 years ago)

Author:

vboxsync

Message:

iprt/asm.h: Cleaned up the ASMMemIsAll8/U32 mess and implmeneted the former in assembly. (Found inverted usage due to bad naming in copyUtf8Block, but it is fortunately an unused method.) Replaces the complicated ASMBitFirstSet based scanning in RTSgBufIsZero with a simple call to the new ASMMemIsZero function.

File:

• trunk/src/VBox/Devices/BiosCommonCode/MakeAlternativeSource.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/BiosCommonCode/MakeAlternativeSource.cpp

@@ -612 +612 @@
                      "  ; Padding %#x bytes at %#x\n", cbPadding, uFlatAddr);
     uint8_t const  *pb = &g_pbImg[uFlatAddr - g_uBiosFlatBase];
-    if (!ASMMemIsAll8(pb, cbPadding, 0))
+    if (ASMMemIsZero(pb, cbPadding))
         return outputPrintf("  times %u db 0\n", cbPadding);
 
@@ -955 +955 @@
         /* Trailing zero padding detection. */
         if (   *pb == '\0'
-            && ASMMemIsAll8(pb, RT_MIN(cb, 8), 0) == NULL)
-        {
-            void    *pv      = ASMMemIsAll8(pb, cb, 0);
+            && ASMMemIsZero(pb, RT_MIN(cb, 8)))
+        {
+            void    *pv      = ASMMemFirstNonZero(pb, cb);
             uint32_t cbZeros = pv ? (uint32_t)((uint8_t const *)pv - pb) : cb;
             if (!outputPrintf("    times %#x db 0\n", cbZeros))