-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🥳 awesome & some rough edges fuond #1
Comments
Yes. When two
Strange. I'll take a look. Something to do with type inference maybe?
Weird, I'll take a look.
There was an
Oops. That should be easy to fix, will do. Thank you for your attempt to use this in a real project! And thanks for great feedback too. I'll iterate on this in the next few days. |
Thank you for the great work and kind response :), even in its current state, I'd roll that out as an opt-in feature for all simd-json internals; it's just that good :D I'm super eager to see the final gains when the string parsing can be iex-ified as well as they happen so darn often o.o edit: I'll try to make minimal examples as a PR for some of the things, it's probably a lot easier to look at then a whole lot of simd-json code ;) |
was a false positive, some other part of the function temporary needed me to change that , I Could roll it back
smae |
First of all, this is some truely inspired and awesome work. Thank you!
I saw it and had to give it a spin with simd-json, the results are great and I haven't even gotten it through everything. You definitely are onto something with thins, especially for applications like json parsing where there are tight loops and small functions with results this seems to do wonders!
While adding iex I ran into a few snags, most of them I could work around one I couldn't. I figured even with this being experimental and new it wouldn't hurt to report them :)
?
andOK
Ok's with complex content doesn't get translatedconstructing an err in a macro isn't recognized and requires inlining?
in macros aren't recognized (this I couldn't work around)The text was updated successfully, but these errors were encountered: