-
-
Couldn't load subscription status.
- Fork 33.6k
deps: update simdjson to 4.0.7 #59883
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
Conversation
|
Review requested:
|
b311b5c to
73ddddb
Compare
|
i think we need to add no-exceptions to test matrix of simdjson |
|
@anonrig We do test without exceptions in various configurations, but, evidently, not sufficiently. A patch is upcoming. |
|
Version 4.0.6 should solve this build errors. See simdjson/simdjson#2464 |
|
Patch 4.0.6 should fix this issue. We were not compiling with The flag is fine by itself, and it should probably be on by default, but it is not, you have to specifically invoke it. But I think it is off by default because it has false positives. In our case, it was a false positive: there were no ambiguities. |
|
I meant to approve a different PR, ignore my approval here. |
73ddddb to
2ba6765
Compare
24e004d to
98ef251
Compare
4b563c1 to
b01105b
Compare
b01105b to
2467f44
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RSLGTM
|
Landed in eea19fa |
PR-URL: #59883 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
This is an automated update of simdjson to 4.0.7.