unisonweb / unison

A friendly programming language from the future
https://unison-lang.org
Other
5.81k stars 271 forks source link

Enable top-level value caching in jit #5438

Closed dolio closed 2 weeks ago

dolio commented 2 weeks ago

This enables the code that will use the top-level caching flag in the jit to generate values that are only evaluated once.

The main change required was upgrading @unison/internal to the latest base to get correctly sorted definitions. Otherwise it was just turning on the code that enables the hint that was already being generated.

Fixes #4495 on the jit.

aryairani commented 2 weeks ago

I didn't mean to merge it, and not sure how it happened... I thought I was leaving a rubber stamp review. That said, I am not opposed to merging it

dolio commented 2 weeks ago

Merging was fine.