4782
90%

Ran 27 Jun 2026 03:17AM UTC

Jobs 1

Files 381

Run time 1min

Badge

Embed ▾

Committed 27 Jun 2026 03:16AM UTC coverage: 90.973%. Remained the same

Build # 4782

Build Type

push

github

Committed by

bobzhang

Commit Message

perf(builtin): make Array::swap inlineable

Follow-up to #3716. `Array::swap` is a small, hot accessor (used throughout
sorting / partitioning / shuffling) that was left out of #3716 because its
abort message carries two indices and could not reuse the single-index
`index_out_of_bounds` helper.

Like the accessors fixed in #3716, its inline interpolated
`abort("... index is (\{i}, \{j})")` builds a `StringBuilder` and inflates the
body past clang's inline-cost threshold (cost=995 vs 225). The backend emits it
as an out-of-line call, so loops that call `swap` cannot inline it.

Add a two-index `index_out_of_bounds2` cold helper (`#inline(never)`) and route
`Array::swap` through it. The body drops to cost ~70 and clang inlines `swap`
into its callers; the exact diagnostic message is preserved.

Validation:
- moon check --target native (clean)
- moon test --target native -p builtin (2835 passed)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

15167 of 16672 relevant lines covered (90.97%)

189010.47 hits per line

Coverage Regressions

Lines	Coverage	∆	File
1	99.56	-0.0%	builtin/array.mbt

Jobs

ID	Job ID	Ran	Files	Coverage
1	4782.1	27 Jun 2026 03:17AM UTC	382	90.99	GitHub Action Run

moonbitlang / core / 4782
90%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Coverage Regressions

Jobs

Source Files on build 4782

moonbitlang / core / 4782 90%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Coverage Regressions

Jobs

Source Files on build 4782

moonbitlang / core / 4782
90%

README BADGES
x