Title: [248171] trunk/Source/_javascript_Core
- Revision
- 248171
- Author
- [email protected]
- Date
- 2019-08-02 12:46:45 -0700 (Fri, 02 Aug 2019)
Log Message
[JSC] Use "destroy" function directly for JSWebAssemblyCodeBlock and WebAssemblyFunction
https://bugs.webkit.org/show_bug.cgi?id=200385
Reviewed by Mark Lam.
These CellTypes are not using classInfo stored in the cells, so we can just call JSWebAssemblyCodeBlock::destroy
and WebAssemblyFunction::destroy directly.
* wasm/js/JSWebAssemblyCodeBlockHeapCellType.cpp:
(JSC::JSWebAssemblyCodeBlockDestroyFunc::operator() const):
* wasm/js/WebAssemblyFunctionHeapCellType.cpp:
(JSC::WebAssemblyFunctionDestroyFunc::operator() const):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (248170 => 248171)
--- trunk/Source/_javascript_Core/ChangeLog 2019-08-02 19:17:52 UTC (rev 248170)
+++ trunk/Source/_javascript_Core/ChangeLog 2019-08-02 19:46:45 UTC (rev 248171)
@@ -1,3 +1,18 @@
+2019-08-02 Yusuke Suzuki <[email protected]>
+
+ [JSC] Use "destroy" function directly for JSWebAssemblyCodeBlock and WebAssemblyFunction
+ https://bugs.webkit.org/show_bug.cgi?id=200385
+
+ Reviewed by Mark Lam.
+
+ These CellTypes are not using classInfo stored in the cells, so we can just call JSWebAssemblyCodeBlock::destroy
+ and WebAssemblyFunction::destroy directly.
+
+ * wasm/js/JSWebAssemblyCodeBlockHeapCellType.cpp:
+ (JSC::JSWebAssemblyCodeBlockDestroyFunc::operator() const):
+ * wasm/js/WebAssemblyFunctionHeapCellType.cpp:
+ (JSC::WebAssemblyFunctionDestroyFunc::operator() const):
+
2019-08-02 Mark Lam <[email protected]>
Gardening: build fix.
Modified: trunk/Source/_javascript_Core/wasm/js/JSWebAssemblyCodeBlockHeapCellType.cpp (248170 => 248171)
--- trunk/Source/_javascript_Core/wasm/js/JSWebAssemblyCodeBlockHeapCellType.cpp 2019-08-02 19:17:52 UTC (rev 248170)
+++ trunk/Source/_javascript_Core/wasm/js/JSWebAssemblyCodeBlockHeapCellType.cpp 2019-08-02 19:46:45 UTC (rev 248171)
@@ -38,7 +38,7 @@
ALWAYS_INLINE void operator()(VM&, JSCell* cell) const
{
static_assert(std::is_final<JSWebAssemblyCodeBlock>::value, "Otherwise, this code would not be correct.");
- JSWebAssemblyCodeBlock::info()->methodTable.destroy(cell);
+ JSWebAssemblyCodeBlock::destroy(cell);
}
};
Modified: trunk/Source/_javascript_Core/wasm/js/WebAssemblyFunctionHeapCellType.cpp (248170 => 248171)
--- trunk/Source/_javascript_Core/wasm/js/WebAssemblyFunctionHeapCellType.cpp 2019-08-02 19:17:52 UTC (rev 248170)
+++ trunk/Source/_javascript_Core/wasm/js/WebAssemblyFunctionHeapCellType.cpp 2019-08-02 19:46:45 UTC (rev 248171)
@@ -38,7 +38,7 @@
ALWAYS_INLINE void operator()(VM&, JSCell* cell) const
{
static_assert(std::is_final<WebAssemblyFunction>::value, "Otherwise, this code would not be correct.");
- WebAssemblyFunction::info()->methodTable.destroy(cell);
+ WebAssemblyFunction::destroy(cell);
}
};
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
