Title: [282832] trunk/Source/_javascript_Core
Revision
282832
Author
[email protected]
Date
2021-09-21 11:43:34 -0700 (Tue, 21 Sep 2021)

Log Message

Replace a few ASSERTs with static_asserts in the ARM64 MacroAssemblers.
https://bugs.webkit.org/show_bug.cgi?id=230569

Reviewed by Yusuke Suzuki.

* assembler/ARM64Assembler.h:
* assembler/ARM64EAssembler.h:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (282831 => 282832)


--- trunk/Source/_javascript_Core/ChangeLog	2021-09-21 18:30:37 UTC (rev 282831)
+++ trunk/Source/_javascript_Core/ChangeLog	2021-09-21 18:43:34 UTC (rev 282832)
@@ -1,3 +1,13 @@
+2021-09-21  Mark Lam  <[email protected]>
+
+        Replace a few ASSERTs with static_asserts in the ARM64 MacroAssemblers.
+        https://bugs.webkit.org/show_bug.cgi?id=230569
+
+        Reviewed by Yusuke Suzuki.
+
+        * assembler/ARM64Assembler.h:
+        * assembler/ARM64EAssembler.h:
+
 2021-09-21  Justin Michaud  <[email protected]>
 
         Differential testing: live statement don't execute

Modified: trunk/Source/_javascript_Core/assembler/ARM64Assembler.h (282831 => 282832)


--- trunk/Source/_javascript_Core/assembler/ARM64Assembler.h	2021-09-21 18:30:37 UTC (rev 282831)
+++ trunk/Source/_javascript_Core/assembler/ARM64Assembler.h	2021-09-21 18:43:34 UTC (rev 282832)
@@ -41,8 +41,8 @@
 #include <zircon/syscalls.h>
 #endif
 
-#define CHECK_DATASIZE_OF(datasize) ASSERT(datasize == 32 || datasize == 64)
-#define CHECK_MEMOPSIZE_OF(size) ASSERT(size == 8 || size == 16 || size == 32 || size == 64 || size == 128);
+#define CHECK_DATASIZE_OF(datasize) static_assert(datasize == 32 || datasize == 64)
+#define CHECK_MEMOPSIZE_OF(size) static_assert(size == 8 || size == 16 || size == 32 || size == 64 || size == 128);
 #define DATASIZE_OF(datasize) ((datasize == 64) ? Datasize_64 : Datasize_32)
 #define MEMOPSIZE_OF(datasize) ((datasize == 8 || datasize == 128) ? MemOpSize_8_or_128 : (datasize == 16) ? MemOpSize_16 : (datasize == 32) ? MemOpSize_32 : MemOpSize_64)
 #define CHECK_DATASIZE() CHECK_DATASIZE_OF(datasize)
@@ -3862,6 +3862,7 @@
 } // namespace JSC
 
 #undef CHECK_DATASIZE_OF
+#undef CHECK_MEMOPSIZE_OF
 #undef DATASIZE_OF
 #undef MEMOPSIZE_OF
 #undef CHECK_DATASIZE

Modified: trunk/Source/_javascript_Core/assembler/ARM64EAssembler.h (282831 => 282832)


--- trunk/Source/_javascript_Core/assembler/ARM64EAssembler.h	2021-09-21 18:30:37 UTC (rev 282831)
+++ trunk/Source/_javascript_Core/assembler/ARM64EAssembler.h	2021-09-21 18:43:34 UTC (rev 282832)
@@ -31,7 +31,7 @@
 
 namespace JSC {
 
-#define CHECK_MEMOPSIZE_OF(size) ASSERT(size == 8 || size == 16 || size == 32 || size == 64 || size == 128);
+#define CHECK_MEMOPSIZE_OF(size) static_assert(size == 8 || size == 16 || size == 32 || size == 64 || size == 128);
 #define MEMOPSIZE_OF(datasize) ((datasize == 8 || datasize == 128) ? MemOpSize_8_or_128 : (datasize == 16) ? MemOpSize_16 : (datasize == 32) ? MemOpSize_32 : MemOpSize_64)
 #define CHECK_MEMOPSIZE() CHECK_MEMOPSIZE_OF(datasize)
 #define MEMOPSIZE MEMOPSIZE_OF(datasize)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to