Remove obsolete multihelpers mode switch#213
Conversation
- Drop `{$modeswitch multihelpers}` from shared compiler settings
- Update build and style docs to match the current Pascal configuration
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
💤 Files with no reviewable changes (3)
📝 WalkthroughWalkthroughThe changes remove a single FreePascal compiler directive Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Benchmark Results274 benchmarks Interpreted: 🟢 83 improved · 🔴 139 regressed · 52 unchanged · avg -1.6% arraybuffer.js — Interp: 🟢 1, 🔴 13 · avg -6.5% · Bytecode: 🟢 6, 🔴 1, 7 unch. · avg +0.4%
arrays.js — Interp: 🔴 17, 2 unch. · avg -7.6% · Bytecode: 🟢 3, 🔴 8, 8 unch. · avg -0.5%
async-await.js — Interp: 🔴 6 · avg -8.8% · Bytecode: 🟢 5, 1 unch. · avg +1.9%
classes.js — Interp: 🔴 31 · avg -8.8% · Bytecode: 🟢 1, 🔴 7, 23 unch. · avg -0.9%
closures.js — Interp: 🔴 11 · avg -3.8% · Bytecode: 🟢 3, 8 unch. · avg +1.2%
collections.js — Interp: 🟢 8, 🔴 3, 1 unch. · avg +3.7% · Bytecode: 🟢 5, 7 unch. · avg +1.3%
destructuring.js — Interp: 🔴 20, 2 unch. · avg -4.2% · Bytecode: 🟢 2, 🔴 4, 16 unch. · avg -0.4%
fibonacci.js — Interp: 🟢 1, 🔴 3, 4 unch. · avg -0.8% · Bytecode: 🟢 4, 🔴 1, 3 unch. · avg +0.0%
for-of.js — Interp: 🟢 5, 2 unch. · avg +2.5% · Bytecode: 🟢 1, 🔴 2, 4 unch. · avg -1.4%
helpers/bench-module.js — Interp: 0 · Bytecode: 0
iterators.js — Interp: 🟢 11, 🔴 1, 8 unch. · avg +1.9% · Bytecode: 🟢 13, 🔴 2, 5 unch. · avg +2.6%
json.js — Interp: 🟢 1, 🔴 10, 9 unch. · avg -2.1% · Bytecode: 🟢 1, 🔴 4, 15 unch. · avg -0.4%
jsx.jsx — Interp: 🟢 5, 🔴 8, 8 unch. · avg -0.7% · Bytecode: 🟢 10, 11 unch. · avg +2.8%
modules.js — Interp: 🟢 9 · avg +5.8% · Bytecode: 🟢 7, 🔴 1, 1 unch. · avg +0.7%
numbers.js — Interp: 🟢 7, 🔴 2, 2 unch. · avg +1.9% · Bytecode: 🟢 7, 🔴 1, 3 unch. · avg +6.3%
objects.js — Interp: 🟢 3, 🔴 3, 1 unch. · avg +3.0% · Bytecode: 🟢 2, 5 unch. · avg +1.3%
promises.js — Interp: 🟢 4, 🔴 4, 4 unch. · avg +1.0% · Bytecode: 🟢 3, 🔴 2, 7 unch. · avg +0.7%
regexp.js — Interp: 🟢 4, 🔴 2, 5 unch. · avg +1.0% · Bytecode: 🔴 4, 7 unch. · avg -0.7%
strings.js — Interp: 🟢 11 · avg +6.3% · Bytecode: 🟢 8, 3 unch. · avg +3.3%
typed-arrays.js — Interp: 🟢 13, 🔴 5, 4 unch. · avg +2.1% · Bytecode: 🟢 14, 8 unch. · avg +3.7%
Measured on ubuntu-latest x64. Benchmark ranges compare cached main-branch min/max ops/sec with the PR run; overlapping ranges are treated as unchanged noise. Percentage deltas are secondary context. |
Suite Timing
Measured on ubuntu-latest x64. |
Summary
{$modeswitch multihelpers}directive from the shared Pascal include file.Testing
Summary by CodeRabbit
Chores
Documentation