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