Skip to content

Conversation

@SyMind
Copy link
Member

@SyMind SyMind commented Oct 28, 2025

This PR adds a new benchmark case for rspack source operations by splitting the existing complex_replace_source benchmark into two separate benchmarks: one for source maps and one for source content.

Key Changes:

  • Split benchmark_complex_replace_source into two distinct benchmark functions
  • Updated imports to reflect the new benchmark function names
@codspeed-hq
Copy link

codspeed-hq bot commented Oct 28, 2025

CodSpeed Performance Report

Merging #187 will not alter performance

Comparing new-rspack-source-bench-case (5e5011c) with main (07c2e81)

Summary

✅ 8 untouched
🆕 2 new
⏩ 5 skipped1

Benchmarks breakdown

Benchmark BASE HEAD Change
🆕 complex_replace_source_map N/A 55.4 ms N/A
🆕 complex_replace_source_source N/A 2.8 ms N/A

Footnotes

  1. 5 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@SyMind SyMind marked this pull request as ready for review October 28, 2025 11:53
Copilot AI review requested due to automatic review settings October 28, 2025 11:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new benchmark case for rspack source operations by splitting the existing complex_replace_source benchmark into two separate benchmarks: one for source maps and one for source content.

Key Changes:

  • Split benchmark_complex_replace_source into two distinct benchmark functions
  • Updated imports to reflect the new benchmark function names

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SyMind SyMind merged commit 5b2f16c into main Oct 28, 2025
13 of 14 checks passed
@SyMind SyMind deleted the new-rspack-source-bench-case branch October 28, 2025 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants