Conversation
|
commit: |
commit: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #137 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 11 11
Lines 1005 1007 +2
=========================================
+ Hits 1005 1007 +2 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
This grows the pre-allocated memory size but provides no noticeable improvement. The indirect effect is that it further compacts the main loop, reverting the #135 change, and the |
| // so the load never depends on the previous iteration's load; a fully | ||
| // fused (state x category) DFA measures 1.4-2x *slower* on V8 for exactly | ||
| // that reason. | ||
| export const BND = new Uint8Array(256 << 5); |
There was a problem hiding this comment.
256 bytes -> 8192 bytes pre-allocated memory, runs almost back to the memory usage allocated in v14.1 (8.5KB) - v14.4 (7.4KB).
It's worth the cost if there are significant advantages in terms of code size or performance.
However, the current library is already compacted very well, and most duplicate code is automatically removed after compression. Smaller code can actually become larger after compression.
I need to check other platforms additionally in terms of performance. Such as Safari and Firefox.
No description provided.