eliott/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird synced 2026-05-03 13:02:09 +02:00
Code Issues Packages Projects Releases Wiki Activity

LibJS: Remove pipeline comparison infrastructure

Browse Source 
Remove PipelineComparison.cpp/h and all LIBJS_COMPARE_PIPELINES
support from RustIntegration.cpp. This includes:

- The compare_pipelines_enabled() function
- All comparison blocks in compile_script/eval/module/function
- The pair_shared_function_data() helper
- The m_cpp_comparison_sfd field on SharedFunctionInstanceData

The Rust pipeline has been validated extensively through comparison
testing and no longer needs the side-by-side verification harness.
This commit is contained in:
Andreas Kling
2026-03-19 12:07:57 -05:00
committed by Andreas Kling
parent 22e8e99d4c
commit 2c45472a11
Notes: github-actions[bot] 2026-03-20 02:57:55 +00:00 
Author: https://github.com/awesomekling
Commit: https://github.com/LadybirdBrowser/ladybird/commit/2c45472a118
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/8517
7 changed files with 17 additions and 330 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Libraries/LibJS/Runtime/SharedFunctionInstanceData.cpp
View File

@@ -299,7 +299,6 @@ void SharedFunctionInstanceData::visit_edges(Visitor& visitor)
{
Base::visit_edges(visitor);
visitor.visit(m_executable);
visitor.visit(m_cpp_comparison_sfd);
for (auto& function : m_functions_to_initialize)
visitor.visit(function.shared_data);
m_class_field_initializer_name.visit([&](PropertyKey const& key) { key.visit_edges(visitor); }, [](auto&) {});
@@ -381,7 +380,6 @@ void SharedFunctionInstanceData::clear_compile_inputs()
m_lexical_bindings.clear();
RustIntegration::free_function_ast(m_rust_function_ast);
m_rust_function_ast = nullptr;
m_cpp_comparison_sfd = nullptr;
}
}