3 ms·
The Register source has this comment: // Registers can't be a record type because the values need to be truncated to 8 bits when writing, so setters are ne
by ibejoeb 6mo ago
The Register source has this comment:
// Registers can't be a record type because the values need to be truncated to 8 bits when writing, so setters are needed
// This is for the web renderer as Fable transpiles uint8 to Number (more than 8 bits) in JS and doesn't apply any truncation
// Known non-standard behaviour in Fable (https://fable.io/docs/javascript/compatibility.html#numeric-types)
So, I think, it's just conservatively cleaning the data due to Fable's widening via js Number on the web target.
- debugnik 6mo agoOof, thanks for pointing that out, I hadn't noticed and I've only ever used F# on .NET. That's terrible on Fable's part, the least they could do is truncate. I wasn't aware Fable's translation is so naive.
- ibejoeb 6mo agoI haven't used Fable much, but apparently it maps .NET arrays to js TypedArray. Presumably you could keep the registers in 8-element array and fable will properly produce a Uint8Array. I'd like to benchmark that.
- rienbdj 6mo agoIt’s really hard to please everyone all of the time on this front. This kind of thing is why Roc compiles to WASM but not JS.
- omcnoe 6mo agoFable is great but it has a surprising number of these hidden behaviour changes that are really hard to detect when writing code against it.