Brian Carroll
Brian Carroll
@Qqwy looks like all this needs is a test, right? Anything else blocking it? I'd love to have a parser in the repo. Happy to help out.
Ok I can make a test, and maybe rename the example directory to "parsers" so we can reuse the library and platform for other formats.
Anyone got any ideas why this won't pass on CI? It passes locally.
OK I got it to fail locally with `cargo test --release -p roc_cli csv`. It crashes in the surgical linker, because `memmove` isn't defined. But why isn't it defined? Other...
> Optimization is relevant because "normal" code can be turned into a memmove by the optimizer. Hmm. Maybe I'm missing something. I can certainly see how optimization _of the Roc...
Actually `roc_memcpy` and `roc_memset` are both defined in the host already and we don't have a `roc_memmove`.
> I guess every platform needs a roc_memmov cause llvm might add that call. That sounds like a simple enough solution, I'll give it a go.
Well, damn. That didn't work.
If you want, I can pair on this with you later. I just finished work a few minutes ago so I need a couple of hours to have dinner and...
By the way I'm impressed with how far you got with this, nice!