Skip to content

Add cached IsoDateTime + some benchmarks - #757

Merged
nekevss merged 2 commits into
mainfrom
add-zdt-caching
Oct 21, 2026
Merged

nekevss merged 2 commits into
mainfrom
add-zdt-caching

Conversation

@nekevss

@nekevss nekevss commented Oct 21, 2026

Copy link
Copy Markdown
Member

Closes #756.

Adds a cached IsoDateTime to ZonedDateTime. This PR also added some benchmarks for the ZonedDateTime methods as well.

@nekevss
nekevss merged commit 7602b09 into main Oct 21, 2026
9 checks passed
@Manishearth
Manishearth deleted the add-zdt-caching branch October 21, 2026 17:06
gesposito added a commit to gesposito/momentum that referenced this pull request Oct 25, 2026
…re filed

oven-sh/bun#46865: Bun 1.4.4's JavaScriptCore rejects EST5EDT, CST6CDT,
MST7MDT and PST8PDT on macOS. fullcalendar/temporal-polyfill#118: the
polyfill identifies zones by Intl's resolvedOptions().timeZone, which
JavaScriptCore no longer canonicalizes. Both rows in docs/upstream.md,
sections 8 and 9 of docs/bun-compatibility.md and the known-failures
comment link them; section 8 no longer cites supportedValuesOf, which
lists only primary zones and so says nothing about links.

The sweep of the other items: temporal_rs merged the ZonedDateTime cache
(boa-dev/temporal#757, closing #756) on 2026-10-21, after v0.2.6, so V8
does not have it yet; the fix for oven-sh/bun#46342 is in review
(oven-sh/WebKit#616).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cache the ISO date-time on ZonedDateTime so its field getters stop recomputing it

2 participants