Title: [273269] trunk/Source/_javascript_Core
Revision
273269
Author
[email protected]
Date
2021-02-22 12:15:51 -0800 (Mon, 22 Feb 2021)

Log Message

Remove unused internal fields from AbstractModuleLoader
https://bugs.webkit.org/show_bug.cgi?id=222256

Reviewed by Saam Barati.

* runtime/AbstractModuleRecord.h:
(JSC::AbstractModuleRecord::initialValues):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (273268 => 273269)


--- trunk/Source/_javascript_Core/ChangeLog	2021-02-22 20:12:46 UTC (rev 273268)
+++ trunk/Source/_javascript_Core/ChangeLog	2021-02-22 20:15:51 UTC (rev 273269)
@@ -1,3 +1,13 @@
+2021-02-22  Keith Miller  <[email protected]>
+
+        Remove unused internal fields from AbstractModuleLoader
+        https://bugs.webkit.org/show_bug.cgi?id=222256
+
+        Reviewed by Saam Barati.
+
+        * runtime/AbstractModuleRecord.h:
+        (JSC::AbstractModuleRecord::initialValues):
+
 2021-02-21  Lauro Moura  <[email protected]>
 
         Fix warning after r273225

Modified: trunk/Source/_javascript_Core/runtime/AbstractModuleRecord.h (273268 => 273269)


--- trunk/Source/_javascript_Core/runtime/AbstractModuleRecord.h	2021-02-22 20:12:46 UTC (rev 273268)
+++ trunk/Source/_javascript_Core/runtime/AbstractModuleRecord.h	2021-02-22 20:15:51 UTC (rev 273269)
@@ -38,10 +38,10 @@
 
 // Based on the Source Text Module Record
 // http://www.ecma-international.org/ecma-262/6.0/#sec-source-text-module-records
-class AbstractModuleRecord : public JSInternalFieldObjectImpl<4> {
+class AbstractModuleRecord : public JSInternalFieldObjectImpl<2> {
     friend class LLIntOffsetsExtractor;
 public:
-    using Base = JSInternalFieldObjectImpl<4>;
+    using Base = JSInternalFieldObjectImpl<2>;
 
     static constexpr bool needsDestruction = true;
 
@@ -57,19 +57,15 @@
 
     enum class Field : uint32_t {
         State,
-        Next,
-        This,
         Frame,
     };
 
-    static_assert(numberOfInternalFields == 4);
+    static_assert(numberOfInternalFields == 2);
     static std::array<JSValue, numberOfInternalFields> initialValues()
     {
         return { {
             jsNumber(static_cast<int32_t>(State::Init)),
             jsUndefined(),
-            jsUndefined(),
-            jsUndefined(),
         } };
     }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to