[klauspost]

Backport of https://go.dev/cl/829570 (golang/go#81398).

When estimating the gain of a candidate match at level 9, findMatch summed the literal code lengths of the whole match for every candidate that was longer than the best so far, which cost about a tenth of the encoding time.

Compute a cumulative literal cost array over the window once per deflateLazy call, right after the literal encoder used for the estimate is generated, so that the cost of any range is a subtraction. Also hoist the hash chain fields into locals for the chain walk.

Unlike upstream, litBits is a lazily allocated slice rather than an array in advancedState, so only level 9 writers pay the 256KB; levels 7 and 8 never generate the estimate tables.

findMatch took the gain path for d.chain >= 100 while deflateLazy only generated the tables it needs for d.chain > 100. No level has a chain of exactly 100 so this was unreachable, but the mismatch is now load bearing (a nil litBits instead of a nil d.h), so the conditions are made to agree.

bitLengthRaw is no longer used and is removed. The output is byte-identical.

goos: windows
goarch: amd64
pkg: github.com/klauspost/compress/flate
cpu: AMD Ryzen 9 9950X 16-Core Processor
                     │    master    │            cl-829570            │
                     │   sec/op    │   sec/op     vs base                │
PR/Digits/Huffman-32   804.3µ ± 1%   801.7µ ± 1%        ~ (p=0.305 n=15)
PR/Digits/L1-32        4.731m ± 1%   4.716m ± 1%        ~ (p=0.683 n=15)
PR/Digits/L4-32        5.518m ± 2%   5.486m ± 2%        ~ (p=0.285 n=15)
PR/Digits/L6-32        7.879m ± 5%   7.875m ± 1%        ~ (p=0.267 n=15)
PR/Digits/L7-32        14.54m ± 3%   14.50m ± 1%        ~ (p=0.116 n=15)
PR/Digits/L9-32        32.37m ± 3%   26.79m ± 6%  -17.23% (p=0.000 n=15)
PR/Twain/Huffman-32    1.051m ± 3%   1.051m ± 2%        ~ (p=0.775 n=15)
PR/Twain/L1-32         4.726m ± 2%   4.757m ± 1%        ~ (p=0.595 n=15)
PR/Twain/L4-32         5.944m ± 1%   5.898m ± 1%        ~ (p=0.187 n=15)
PR/Twain/L6-32         7.773m ± 3%   7.832m ± 2%        ~ (p=0.267 n=15)
PR/Twain/L7-32         17.70m ± 2%   17.50m ± 1%        ~ (p=0.202 n=15)
PR/Twain/L9-32         36.68m ± 1%   34.58m ± 2%   -5.72% (p=0.000 n=15)
geomean                6.898m        6.745m        -2.21%

                     │     B/s      │     B/s       vs base                │
PR/Digits/Huffman-32   1.158Gi ± 1%   1.162Gi ± 1%        ~ (p=0.305 n=15)
PR/Digits/L1-32        201.6Mi ± 1%   202.2Mi ± 1%        ~ (p=0.683 n=15)
PR/Digits/L4-32        172.8Mi ± 2%   173.8Mi ± 2%        ~ (p=0.281 n=15)
PR/Digits/L6-32        121.0Mi ± 5%   121.1Mi ± 1%        ~ (p=0.263 n=15)
PR/Digits/L7-32        65.58Mi ± 3%   65.76Mi ± 1%        ~ (p=0.116 n=15)
PR/Digits/L9-32        29.47Mi ± 3%   35.60Mi ± 5%  +20.81% (p=0.000 n=15)
PR/Twain/Huffman-32    907.7Mi ± 3%   907.3Mi ± 2%        ~ (p=0.775 n=15)
PR/Twain/L1-32         201.8Mi ± 2%   200.5Mi ± 1%        ~ (p=0.602 n=15)
PR/Twain/L4-32         160.4Mi ± 1%   161.7Mi ± 1%        ~ (p=0.187 n=15)
PR/Twain/L6-32         122.7Mi ± 3%   121.8Mi ± 2%        ~ (p=0.267 n=15)
PR/Twain/L7-32         53.87Mi ± 2%   54.49Mi ± 1%        ~ (p=0.202 n=15)
PR/Twain/L9-32         26.00Mi ± 1%   27.57Mi ± 2%   +6.05% (p=0.000 n=15)
geomean                138.3Mi        141.4Mi        +2.26%

Original-Author: Alex Gaynor alex.gaynor@gmail.com

Summary by CodeRabbit

• Performance
  • Improved compression decisions at higher compression levels by more accurately evaluating literal and match costs.
  • Accelerated match evaluation during compression, which may improve compression speed while preserving output quality.
  • Improved hash-chain traversal efficiency for compression workloads.
• Compatibility
  • No changes to exported or public interfaces.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 1520efbe-238f-48f8-8868-0e3b371e9c58

📥 Commits

Reviewing files that changed from the base of the PR and between 7dc45b1 and 1b08099.

📒 Files selected for processing (2)

• flate/deflate.go
• flate/huffman_code.go

💤 Files with no reviewable changes (1)

• flate/huffman_code.go

Included review availability: Your plan provides up to 8 included reviews per hour; 4 remain after this review.

---

📝 Walkthrough

Walkthrough

The advanced deflate path now caches cumulative literal Huffman costs. findMatch uses these costs and cached hash-chain references to estimate match gains, including extra bits. The obsolete raw bit-length helper was removed.

Changes

Deflate gain estimation

Layer / File(s)	Summary
Literal cost preparation flate/deflate.go	Advanced deflate state stores cumulative literal costs and builds them for the pending deflateLazy window.
Match gain search flate/deflate.go, flate/huffman_code.go	findMatch estimates gains from cached literal costs and match extra bits. Hash-chain traversal uses cached references. The gain path now includes chain == 100, and bitLengthRaw was removed.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Refactor

Merge Risk: ⚪ Minimal · up to 1b080

The optimization preserves the established deflate gain-estimation contract without an identified correctness, stability, or output-compatibility regression.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: precomputing literal costs for findMatch.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch cl-829570

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}