ci: bump actions/cache/save from 4.3.0 to 6.1.0 - #1
Open
dependabot[bot] wants to merge 1 commit into
Open
Conversation
Bumps [actions/cache/save](https://github.com/actions/cache) from 4.3.0 to 6.1.0. - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](actions/cache@0057852...55cc834) --- updated-dependencies: - dependency-name: actions/cache/save dependency-version: 6.1.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Author
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
Ttimmahlax
pushed a commit
that referenced
this pull request
Aug 23, 2026
Sixty `div` instructions removed, and the shipped object now contains zero `div`/`idiv`, zero `divsd`/`divss`, zero 128-bit soft-division libcalls, and zero magic-multiply divisions by `size_of::<Page>()`. Verified in the disassembly rather than inferred from source — which mattered, because a source-level scan cleared 21 sites that were emitting a three-instruction `movabs; mul; shr` apiece. The load-bearing changes: * `page_index` becomes a shift. `Page::area` already holds `seg + idx * SEGMENT_SLICE_SIZE` and `SEGMENT_SLICE_SIZE` is 2^16, so the slot index needs no divide by 88 and no new field — closing `opps.md` #1 at zero header cost, where the entry had proposed spending 1 KiB of segment header on a spare `u16`. * `slice_offset` counts slices, not bytes. Bytes were justified by `page_of` reading the field on every free; the `page_off` table took that over, and in release the only arithmetic reader left was the one this divides in. * `page_extend`'s batch bound is a shift of `reserved`. Two earlier attempts lost because they kept the computation and only changed its form, moving a live value onto `malloc_generic`'s fast path. * `div_by_block_size` is a branchless reciprocal table. The four-arm match emitted a compare tree plus three magic multiplies, nine of whose instructions were dead on any given call. `free_general` 148 -> 112 instructions, `usable_size_slow` 93 -> 55. * Windows `clock_now` no longer calls `__udivti3`: the QPC frequency is fixed per process and divides 1e9 exactly on every TSC-backed Windows since 8, so the cached quotient turns the conversion into one multiply. * The benchmark harness drops to zero float divisions. Fixes a latent defect found on the way: `huge_alloc` builds its single page slot by hand and never wrote `Page::area`, leaving it null. Nothing read it before; `page_index` and `unalign` both do now, and huge pages set `SINGLE_BLOCK` so they do not take `unalign`'s early return. `bins::is_aligned_to` is public because `rusty_alloc-ffi` uses it and it is total. `div_by_block_size` and `exact_div_by_block_size` are `pub(crate)`: both carry preconditions and return a wrong value rather than diverging outside them. cfrac allocator instructions 3,445,599,360 -> 3,445,578,293. All 13 opscan ops remain below mimalloc; `free` holds at 21 instructions. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Ttimmahlax
pushed a commit
that referenced
this pull request
Aug 23, 2026
`docs/plans/fast-trans.md` carries the full ledger: every site in the inventory with either the win and its measurement, or the counted reason no instruction exists to remove. Two sites claimed as "a shift" turned out to emit nothing at all, and three claimed at a floor are now a counted fact rather than a source-level judgement. It also records four premature stops, because the corrections are the transferable part. Each was a plausible sentence the disassembly contradicted: that an Ir delta measures code which never executes; that "no `div` emitted" means no division; that a constant division is already optimal when the dividend is a known multiple; and that a last-ulp float difference is a reason to decline on a figure printed to two decimals. `opps.md` #1 is closed with the mechanism, including the part the entry got wrong — no spare `Page` field was needed, so the 1 KiB of header it logged as the risk was never spent. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps actions/cache/save from 4.3.0 to 6.1.0.
Release notes
Sourced from actions/cache/save's releases.
... (truncated)
Changelog
Sourced from actions/cache/save's changelog.
... (truncated)
Commits
55cc834Merge pull request #1768 from jasongin/readonly-cached8cd72fBump@actions/cacheto v6.1.0 - handle cache write error due to RO token2c8a9bdMerge pull request #1760 from actions/samirat/esm_migration_and_package_updatee9b91fdPrettier fixese4884b8Rebuild dist10baf01Fixed licensese39b386Fix test mock return orderb692820PR feedback6074912Rebuild dist bundles as ESM to match type:module5a912e8Fix lint and jest issuesDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)