Title: [240526] branches/safari-607-branch
Revision
240526
Author
[email protected]
Date
2019-01-25 16:02:43 -0800 (Fri, 25 Jan 2019)

Log Message

Cherry-pick r240335. rdar://problem/47494779

    2019-01-23  David Kilzer  <[email protected]>

    [JSC] Duplicate global variables: JSC::opcodeLengths
    <https://webkit.org/b/193714>
    <rdar://problem/47340200>

    Reviewed by Mark Lam.

    * bytecode/Opcode.cpp:
    (JSC::opcodeLengths): Move array implementation here and mark
    const.
    * bytecode/Opcode.h:
    (JSC::opcodeLengths): Change to extern declaration.

Modified Paths

Property Changed

Diff

Index: branches/safari-607-branch =================================================================== --- branches/safari-607-branch 2019-01-25 23:52:51 UTC (rev 240525) +++ branches/safari-607-branch 2019-01-26 00:02:43 UTC (rev 240526)

Property changes: branches/safari-607-branch


Modified: svn:mergeinfo

-/trunk:53455,239940,240329 \ No newline at end of property +/trunk:53455,239940,240329,240335 \ No newline at end of property

Modified: branches/safari-607-branch/Source/_javascript_Core/ChangeLog (240525 => 240526)


--- branches/safari-607-branch/Source/_javascript_Core/ChangeLog	2019-01-25 23:52:51 UTC (rev 240525)
+++ branches/safari-607-branch/Source/_javascript_Core/ChangeLog	2019-01-26 00:02:43 UTC (rev 240526)
@@ -1,5 +1,23 @@
 2019-01-25  Mark Lam  <[email protected]>
 
+        Cherry-pick r240335. rdar://problem/47494779
+
+    2019-01-23  David Kilzer  <[email protected]>
+
+            [JSC] Duplicate global variables: JSC::opcodeLengths
+            <https://webkit.org/b/193714>
+            <rdar://problem/47340200>
+
+            Reviewed by Mark Lam.
+
+            * bytecode/Opcode.cpp:
+            (JSC::opcodeLengths): Move array implementation here and mark
+            const.
+            * bytecode/Opcode.h:
+            (JSC::opcodeLengths): Change to extern declaration.
+
+2019-01-25  Mark Lam  <[email protected]>
+
         Cherry-pick r240329. rdar://problem/47458354
 
         This is for a follow up fix that was missed in the cherry-pick for rdar://problem/47458354.

Modified: branches/safari-607-branch/Source/_javascript_Core/bytecode/Opcode.cpp (240525 => 240526)


--- branches/safari-607-branch/Source/_javascript_Core/bytecode/Opcode.cpp	2019-01-25 23:52:51 UTC (rev 240525)
+++ branches/safari-607-branch/Source/_javascript_Core/bytecode/Opcode.cpp	2019-01-26 00:02:43 UTC (rev 240526)
@@ -40,6 +40,12 @@
 
 namespace JSC {
 
+const unsigned opcodeLengths[] = {
+#define OPCODE_LENGTH(opcode, length) length,
+    FOR_EACH_OPCODE_ID(OPCODE_LENGTH)
+#undef OPCODE_LENGTH
+};
+
 const char* const opcodeNames[] = {
 #define OPCODE_NAME_ENTRY(opcode, size) #opcode,
     FOR_EACH_OPCODE_ID(OPCODE_NAME_ENTRY)

Modified: branches/safari-607-branch/Source/_javascript_Core/bytecode/Opcode.h (240525 => 240526)


--- branches/safari-607-branch/Source/_javascript_Core/bytecode/Opcode.h	2019-01-25 23:52:51 UTC (rev 240525)
+++ branches/safari-607-branch/Source/_javascript_Core/bytecode/Opcode.h	2019-01-26 00:02:43 UTC (rev 240526)
@@ -71,16 +71,8 @@
 #undef OPCODE_ID_ENUM
 #endif
 
-IGNORE_WARNINGS_BEGIN("unused-variable")
+extern const unsigned opcodeLengths[];
 
-#define OPCODE_LENGTH(opcode, length) length,
-    static unsigned opcodeLengths[] = {
-        FOR_EACH_OPCODE_ID(OPCODE_LENGTH)
-    };
-#undef OPCODE_LENGTH
-
-IGNORE_WARNINGS_END
-
 #define OPCODE_ID_LENGTHS(id, length) const int id##_length = length;
     FOR_EACH_OPCODE_ID(OPCODE_ID_LENGTHS);
 #undef OPCODE_ID_LENGTHS
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to