Incorrectly parsed escaped strings?

[mcseemk]

Incorrectly parsed escaped strings?

[mcseemk]

This branch works fine for me. Tried stable/beta/nightly on AMD Ryzen 7950X and Graviton 1/2 - everything OK.

[Licenser]

Hi I can't reproduce this with the current version of simd-json and AVX2, can you share some details on version & CPU architecture you're trying this on?

[mcseemk]

Hi, I figured out that it is not arch-related.

If compiled in release mode on the last nightly (rustc 1.73.0-nightly (6ef7d16be 2023-08-19)) it fails everywhere I tried: Graviton1/2, AMD Epyc 7551P, AMD Ryzen 7950X. In debug mode works fine though.

On the last stable rustc 1.71.1 (eb26296b5 2023-08-03) it works fine in both modes.

[Licenser]

Odd,

this works for me on both nightly and test in release mode:

$ cargo +nightly run --release
   Compiling simd-test v0.1.0 (/Users/heinz/Projects/simd-test)
    Finished release [optimized] target(s) in 2.17s
     Running `target/release/simd-test`
simd-json decoded string: {"arg":"test"}
serde-json decoded string: {"arg":"test"}

 $ cargo run --release
    Finished release [optimized] target(s) in 0.22s
     Running `target/release/simd-test`
simd-json decoded string: {"arg":"test"}
serde-json decoded string: {"arg":"test"}

[mcseemk]

Try adding the following to the Cargo.toml:

[profile.release]
codegen-units = 1

Then it should start failing. Bizzarre indeed.

[Licenser]

Ah yes, that does it, but only for nightly. That's very likely a bug in nightly not simd-json but it's a good catch, perhaps best to report it to rust. I updated the repo if you want to link it.

[mcseemk]

I filed the bug, but the response was discouraging. You can check there.

[Licenser]

could you give this branch a try: #308

[Licenser]

Fix is merged to 0.10.6, I'm closing the ticket, if there are more problems feel free to re-open or make a new one :) thanks for the report.