Title: [263277] trunk/Source/_javascript_Core
Revision
263277
Author
[email protected]
Date
2020-06-19 11:07:16 -0700 (Fri, 19 Jun 2020)

Log Message

functionCpuClflush checks that the second argument is Int32 but it actually expects it to be UInt32
https://bugs.webkit.org/show_bug.cgi?id=213388

Patch by Tuomas Karkkainen <[email protected]> on 2020-06-19
Reviewed by Saam Barati.

This changes the check from isInt32() to isUInt32() so that the logic is consistent.

* tools/JSDollarVM.cpp:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (263276 => 263277)


--- trunk/Source/_javascript_Core/ChangeLog	2020-06-19 17:15:00 UTC (rev 263276)
+++ trunk/Source/_javascript_Core/ChangeLog	2020-06-19 18:07:16 UTC (rev 263277)
@@ -1,3 +1,14 @@
+2020-06-19  Tuomas Karkkainen  <[email protected]>
+
+        functionCpuClflush checks that the second argument is Int32 but it actually expects it to be UInt32
+        https://bugs.webkit.org/show_bug.cgi?id=213388
+
+        Reviewed by Saam Barati.
+
+        This changes the check from isInt32() to isUInt32() so that the logic is consistent.
+
+        * tools/JSDollarVM.cpp:
+
 2020-06-18  Mark Lam  <[email protected]>
 
         Unify Bitmap math loops in MarkedBlock::Handle::specializedSweep().

Modified: trunk/Source/_javascript_Core/tools/JSDollarVM.cpp (263276 => 263277)


--- trunk/Source/_javascript_Core/tools/JSDollarVM.cpp	2020-06-19 17:15:00 UTC (rev 263276)
+++ trunk/Source/_javascript_Core/tools/JSDollarVM.cpp	2020-06-19 18:07:16 UTC (rev 263277)
@@ -1742,7 +1742,7 @@
 #if CPU(X86_64) && !OS(WINDOWS)
     VM& vm = globalObject->vm();
 
-    if (!callFrame->argument(1).isInt32())
+    if (!callFrame->argument(1).isUInt32())
         return JSValue::encode(jsBoolean(false));
 
     auto clflush = [] (void* ptr) {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to