Commit Graph
4729 Commits
Author SHA1 Message Date
Chris Tate 97c3b04f73 Merge main and add the Node 24 util batch 2026-10-24 18:55:14 -05:00
Chris Tate e5cdc68bab Store AbortController and AbortSignal handles in class fields (#2125) 2026-10-24 14:32:15 -05:00
Chris Tate 9cb8ed3ce5 Map resolved empty mapped types as the empty object type (#2127) 2026-10-24 14:25:04 -05:00
Chris Tate 7499f9bd39 Keep versioned loops' slow copies free of speculative integer copies (#2137) 2026-10-24 14:14:40 -05:00
Chris Tate 4efcbbdfe4 Report inferred shapes for untyped npm JavaScript (#2036) 2026-10-24 13:49:30 -05:00
Chris Tate 3f03c90e86 Keep versioned loops' slow copies free of speculative integer copies
The copy of a dense-array, byte-window, counted, callback or typed-array
versioned loop that runs when its guard fails no longer gives its loops
speculative integer or int32 kind copies, which emitted each loop again for
a path that rarely runs. Proven integer loops, which emit one copy, are kept
in it.
2026-10-24 13:47:17 -05:00
Chris Tate bf02b9d7cc Write a number-keyed Map update through the entry its read found (#2130) 2026-10-24 13:38:20 -05:00
Chris Tate 3896957230 Report JSON.parse syntax errors with V8's messages and positions (#2129) 2026-10-24 13:34:12 -05:00
Chris Tate c33d693775 Answer instanceof against the primitive wrapper constructors (#2126) 2026-10-24 13:28:00 -05:00
Chris Tate 9dfaddf7bc Check unchanged frontend inputs by file identity before hashing them (#2128) 2026-10-24 13:24:46 -05:00
Chris Tate b78520559f Follow values through shadowed members, unknown keys on arrays, JSON callbacks and proxies
- Named loads also read what was written under that name or an unknown key where an own field
  shadows a builtin: an instance's constructor, a function's call, apply, bind, prototype, name
  and length, a class's prototype and name, an array's methods. Named loads of arrays read what
  was written under unknown keys, which computed stores on arrays now record.
- Computed stores into builtins, promises and collections escape, as named stores there do.
- JSON.stringify calls toJSON on arrays, functions and classes, and on values written under
  unknown keys. A JSON.parse reviver's results are part of the result and of later calls' values.
- Reflect.construct(Proxy, ...) and globalThis.Proxy record a proxy operation, as new Proxy does.
2026-10-24 13:20:02 -05:00
Chris Tate 7ca5bd465b Keep while-loop counters bounded by a typed array length in integers (#2124) 2026-10-24 13:08:15 -05:00
Chris Tate 94c650a9c2 Write a number-keyed Map update through the entry its read found
m.set(k, f(m.get(k))) and a get followed by a set of the same key, with a
number or boolean value, now reuse the entry the read resolved, as
string-keyed maps already do: the write overwrites the value in place
when the map is the same and the key has the read's bits, and takes the
ordinary set otherwise. Before, the write probed the table again: the two
lookups were not merged, because each sits behind its own direct-index
test.
2026-10-24 13:07:38 -05:00
Chris Tate 50b7c806ce Check unchanged frontend inputs by file identity before hashing them
A warm native build validates its frontend cache entry by reading and
SHA-256 hashing every source and declaration file the frontend read.

A tracked read now also records the file's identity (device, inode,
size, modification and change time) when the identity is the same before
and after the read and the file had been unchanged for two seconds. A
later check accepts a file whose identity is unchanged without reading
it, and hashes it otherwise. A write always gives a file a newer change
time, which tools cannot set back, so an unchanged identity means the
content hashed with it. The equivalent-edit check uses the same identity
for the unchanged sources.
2026-10-24 12:52:54 -05:00
Chris Tate 0ea0707861 Reuse dev builds' frontend output after edits that keep token positions (#2123) 2026-10-24 12:51:02 -05:00
Malte UblandClaude Opus 5.5 864feec4cf Map resolved empty mapped types as the empty object type
A mapped type with no members was refused (SC2001) to keep `Partial<T>`
inside a generic body, where keyof T is still unknown, from interning a
wrong empty record. That also refused mapped types the checker resolved
from concrete inputs whose key filter keeps no key: `Record<never, T>`,
`Partial<{}>`, and a catalog type such as
`{ [K in KeyOf<Data, "image"> as Id<K>]: Model }` for data without image
models. When the mapped type comes through an alias whose arguments mention
no type parameter (or a module-level alias without arguments), it is the
empty object type and lowers like `{}`.

pi-ai declares its per-provider image and classifier model catalogs this
way; most providers have none, which refused 105 declaration groups.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-24 10:50:44 -07:00
Malte UblandClaude Opus 5.5 b11cd04ce9 Answer instanceof against the primitive wrapper constructors
`value instanceof Boolean` (or Number, String, BigInt, Symbol) died at run
time in JavaScript with Node's TypeError for a function without a prototype
object, because those constructors are compiled as plain conversion
closures; in TypeScript the right-hand side was refused (SC1090). A
compiled program never creates a boxed primitive (`new Boolean(...)` has
no lowering) and a primitive is never an instance, so the test is false
for every value. It now folds to false after evaluating the operand.

typebox's value guards (IsBoolean, IsNumber, IsString) test this way, so
pi's tool-schema construction and validation died at its first guard.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-24 10:50:43 -07:00
Malte UblandClaude Opus 5.5 684a0abd59 Store AbortController and AbortSignal handles in class fields
A class field typed AbortController or AbortSignal (plain, optional,
nullable, or in an array) maps to checked-dynamic storage, and the
class-field rule refused it (SC1090) although the same handles already
live in locals, Map values, and closures. They now take the checked-dynamic
slot, like the fetch Headers, Request, and Response handles.

controller?.abort() on a nullable controller was refused as a method call
(also for locals): the static abort lowering ran only without an optional
chain. It now runs again once the chain has narrowed the receiver.

Found by pi's TUI Editor (autocompleteAbort: AbortController | undefined).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-24 10:50:42 -07:00
Chris Tate 8796e91f5e Keep while-loop counters bounded by a typed array length in integers 2026-10-24 12:37:29 -05:00
Chris Tate 71646287b2 Divide one-limb BigInt quotients in one pass and chain limb carries (#2121) 2026-10-24 12:30:30 -05:00
Chris Tate 31d3ba07ca Reuse dev builds' frontend output after edits that keep token positions
A native build reuses its stale frontend cache entry when every changed
source is token-equivalent to the text the entry was built from, but dev
builds never did: their debug information records source lines and
columns, which a token-equivalent edit can move.

Builds with debug information (dev builds without --strip) now reuse the
entry when every changed source is token-equivalent and every token keeps
its line and column (tokenPositionEquivalentSource): edits to a comment's
text within its line, comments and whitespace after the last token, and
trailing whitespace. Edits that move a token still rebuild.

The token scanner also reads a `/` after a non-null assertion as division
(`data[i]! / 7`). It used to read it as the start of a regular expression
and give up on the file, so comment edits in files with that pattern
always rebuilt. A `!` after a line break, or after an operator or a
keyword such as `return`, still starts a negated regular expression, and
a `!` after `)`, `}`, `++` or `--` gives up.
2026-10-24 12:22:19 -05:00
Chris Tate aa4e046046 Emit function bodies on up to eight workers (#2118) 2026-10-24 12:04:33 -05:00
Chris Tate ff70c33e38 Divide one-limb BigInt quotients in one pass and chain limb carries
Dividing BigInts of the same limb count, where the quotient is one limb,
takes Knuth's quotient estimate from the top limbs as they read
normalized and settles it with one pass of a - q * b over the operands,
instead of copying both operands shifted and running algorithm D. The
remainder, when wanted, is that pass's result.

The limb loops for add, subtract, multiply by a limb, and multiply-add
and multiply-subtract by a limb process four limbs a pass, with the
carries in add-with-carry chains (__builtin_addcll and __builtin_subcll
where the compiler has them).
2026-10-24 11:59:33 -05:00
Chris Tate fe394f8c30 Keep an unchanged executable on its first Linux rebuild (#2112) 2026-10-24 11:57:52 -05:00
Malte UblandClaude Opus 5.5 d037c66e8c Report JSON.parse syntax errors with V8's messages and positions
JSON.parse's SyntaxError messages were shaped like V8's but differed from
Node's in several ways:
- Every positional message now ends with V8's "(line L column C)", where
  \n, \r and \r\n end lines.
- Positions are UTF-16 offsets, as in Node, not byte offsets.
- An object or array cut off after '{', a property name, a value or an
  array element reports the token V8 expected ("Expected property name or
  '}'", "Expected ':' after property name", "Expected ',' or '}' after
  property value", "Expected ',' or ']' after array element") instead of
  "Unexpected end of JSON input".
- After a comma in an object the message is "Expected double-quoted
  property name".
- An unterminated string reports the end of the input, and a truncated
  \u escape is a "Bad Unicode escape" there.
- A leading zero followed by a digit is an "Unexpected number".
- A misspelled literal names the first differing character, or the end of
  the input.
- An unexpected character is named with the source as V8 does: whole when
  it is under 21 units, otherwise ten units of context on each side; the
  sources "NaN", "Infinity", "undefined" and "[object Object]" name only
  themselves; a quote or a number there is an "Unexpected string" or
  "Unexpected number".

The new corpus program 7993-json-parse-error-messages pins 76 inputs
against Node, including non-ASCII text before the error, line breaks, and
long sources with split surrogate pairs at the context edges.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-24 09:55:05 -07:00
Chris Tate 40f0b6d9e0 Refresh the native compiler's optimization profile (#2120) 2026-10-24 11:51:12 -05:00
Chris Tate 557daa3e9d Refresh the native compiler's optimization profile
The committed profile was collected before about 200 commits that changed
12,800 lines of the compiler. Functions whose control flow changed since are
optimized as without a profile, so its gain had mostly gone: a cold tsc-ts
speed build was 2.7% faster with it than with no profile, against 5% to 6%
with a fresh one.

Regenerated with scripts/refresh-cli-profile.mjs on this commit.
2026-10-24 11:47:19 -05:00
Chris Tate c09385c7ab Emit function bodies on up to eight workers
The native compiler forked four worker processes to emit the function
bodies of a large module. It now forks one per available core, up to
eight. SCRIPTC_EMIT_WORKERS still sets the count. The output is the same
text at any count.
2026-10-24 11:29:56 -05:00
Chris Tate 022428ead6 Write the first stdin chunk after both readers start in the 'readable' EOF test (#2109) 2026-10-24 10:30:31 -05:00
Chris Tate 205f099957 Keep an unchanged executable on its first Linux rebuild
On Linux the executable cache stored an executable only from a key's second
link on, so the first unchanged rebuild after a build linked again, and only
the build after that restored the executable.

- A key's first link on Linux is traced, in the same verbose link. Instead
  of storing the executable, the cache records the output's file identity
  together with the link's inputs once the output is in place.
- keep() answers an unchanged rebuild whose output still has that identity
  and whose inputs are unchanged. The build then neither links nor copies:
  the output stays as it is.
- A rebuild whose output changed or is missing links again; from that second
  link on, the executable is stored as before and restored where needed.
- The cache records the output after every Linux build that places one, so
  the next unchanged rebuild keeps it.

darwin keeps its existing behavior.
2026-10-24 10:26:38 -05:00
Chris Tate 7406ec63f4 Read the node index in more LLVM emission analyses (#2108) 2026-10-24 10:19:42 -05:00
Chris Tate 72eb9318bf Analyze int32 slots in queue order, after fixing an order-dependent narrowing (#2107) 2026-10-24 10:02:34 -05:00
Chris Tate 38730910c8 Write the first stdin chunk after both readers start in the 'readable' EOF case
A reader that starts after the second chunk arrives reads both chunks in
one 'readable' event. On loaded test workers Node started that late while
the compiled program did not, so the two outputs differed. The first chunk
now waits 500 ms.
2026-10-24 09:56:15 -05:00
Chris Tate aa1c26845c Cache untyped property access in programs that declare classes (reland) (#2102) 2026-10-24 09:55:54 -05:00
Chris Tate 648e8e05c6 Read the node index in more LLVM emission analyses
Several analyses the emitter runs per function or per module walked whole
bodies to collect facts from a few node kinds. They now read those kinds
from the module's node index (ir/node-index.ts), which records every node
once in walk order, and the output is unchanged.

- numberGlobalWriters, readsLoopMetrics, closure creation sites
  (LazyCaptures) and the ambient receiver readers visit only the kinds
  they inspect.
- findLoopArrayBorrows visits only loop statements, and findIntegerViews
  returns at once for a function without a loop, where it wants no view.
- foldConstantGlobals records which candidates each function reads during
  its write walk instead of walking every function again.
2026-10-24 09:50:02 -05:00
Chris Tate 580e8ac723 Analyze int32 slots in queue order, after fixing an order-dependent narrowing 2026-10-24 09:34:59 -05:00
Chris Tate e579f62556 Inline the array release fast path in speed builds (#2104) 2026-10-24 09:18:35 -05:00
Chris Tate 15e8c0cfac Place each source file's functions in the same partitions (#2083) 2026-10-24 09:08:38 -05:00
Chris Tate 905a290817 Copy typed array sets of the same element kind on shared memory as bytes (#2094) 2026-10-24 09:07:57 -05:00
Chris Tate 6aec284766 Inline the array release fast path under --optimization=speed
Speed builds now release arrays through the inline fast path the other
RC families use: an array that stays alive and has no cycle header is
decremented in place, and only the last release or a headered array
calls scr_arr_release.
2026-10-24 08:50:22 -05:00
Chris Tate 0dab283512 Cut repeated work in LLVM module emission (#2097) 2026-10-24 08:41:40 -05:00
Chris Tate 427e383b5e Cache untyped property access in programs that declare classes (reland)
Reapplies #2017, which #2038 reverted after effect's observability-otlp
fixture crashed in plain builds. The crash was the candidate-buffer
corruption that the previous commit fixes: the cached property sites
change when the collector runs, and with enough environment variables a
pass buffered a string box that was later pooled while still buffered.
2026-10-24 08:22:30 -05:00
Chris Tate 6a8c7056ca Run the int32 slot analysis once per module the emitter emits (#2086) 2026-10-24 08:18:58 -05:00
Chris Tate cc27555171 Dispatch untyped calls on plain arrays without the method ladder (#2085) 2026-10-24 08:18:16 -05:00
Chris Tate 21570a963c Cut repeated work in LLVM module emission
Emitting the LLVM text of large programs spent seconds rebuilding the same
sets, rescanning the finished text and walking every function for passes
that change a few of them. The output is unchanged.

- computeMayThrow builds the set of temporal-dead-zone globals once per
  module instead of copying it into a new set for every function.
- limitInlineReleases finds the release sites of every family in one walk
  over the module text, and rewrites only the parts that hold a function
  over its limit.
- The mentioned-function and node:test checks of parking analysis scan the
  text with indexOf instead of global regular expressions.
- resolveSelfCaptures rewrites only the functions it changes,
  specializeCallbackCalls appends to its write lists instead of copying
  them and counts nodes in a walk it already makes, and CaptureFlow reuses
  one callback for nested statements.
2026-10-24 07:07:20 -05:00
Chris Tate 6a4a1bdec9 Collect the supplied files' global names only for programs with scripts (#2092) 2026-10-24 06:46:07 -05:00
Chris Tate 38c30d2e91 Merge main 2026-10-24 06:45:47 -05:00
Chris Tate fe3f52d14c Merge main 2026-10-24 06:44:55 -05:00
Chris Tate b17268ff74 Take number and string boxes out of the candidate buffers before pooling them (#2091) 2026-10-24 06:30:38 -05:00
Chris Tate 314024a8c7 Copy typed array sets of the same element kind on shared memory as bytes
TypedArray.prototype.set with a SharedArrayBuffer view as its source or
target converted every element through a double, even when both arrays
had the same element kind. That cost about 5 ns per byte, and it changed
NaN payloads: a Float32Array signaling NaN came back quieted (7fa00001
became 7fe00001), where Node copies the bytes.

The same element kind now copies the source's bytes with the relaxed
shared stores. A source that overlaps the target range, a view of the
same buffer in a program without workers, is copied out first. Different
element kinds still convert each element.
2026-10-24 06:29:47 -05:00