icu4x icon indicating copy to clipboard operation
icu4x copied to clipboard

move to wasmer 3.1.1

Open pdogr opened this issue 2 years ago • 7 comments

#3241 cargo audit still complains about mach

Crate:     mach
Version:   0.3.2
Warning:   unmaintained
Title:     mach is unmaintained
Date:      2020-07-14
ID:        RUSTSEC-2020-0168
URL:       https://rustsec.org/advisories/RUSTSEC-2020-0168
Dependency tree:
mach 0.3.2
├── wasmer-vm 3.1.1
│   ├── wasmer-compiler 3.1.1
│   │   ├── wasmer-compiler-singlepass 3.1.1
│   │   │   └── wasmer 3.1.1
│   │   │       ├── wasmer-wasi-types 3.1.1
│   │   │       │   └── wasmer-wasi 3.1.1
│   │   │       │       └── icu_codepointtrie_builder 0.3.5
│   │   │       │           └── icu_datagen 1.2.0
│   │   │       │               ├── tutorials-test 0.0.0
│   │   │       │               └── icu_testdata_scripts 0.0.0
│   │   │       ├── wasmer-wasi 3.1.1
│   │   │       └── icu_codepointtrie_builder 0.3.5
│   │   └── wasmer 3.1.1
│   └── wasmer 3.1.1
└── region 3.0.0
    ├── wasmer-vm 3.1.1
    └── wasmer-compiler 3.1.1

pdogr avatar Apr 20 '23 19:04 pdogr

Notice: the branch changed across the force-push!

  • Cargo.lock is different
  • components/collections/codepointtrie_builder/src/wasm.rs is different

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

Notice: the branch changed across the force-push!

  • Cargo.lock is different

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

Some benches on Mac x86 Command run

cargo run --release --bin=icu4x-datagen --all-features -- --icuexport-root=../repodata/data/icuexport --syntax=postcard --out=/dev/null --format=blob --overwrite --keys=segmenter/word@1 --all-locales

This PR

INFO  [icu_datagen] Writing key: segmenter/word@1
INFO  [icu_provider_blob::export::blob_exporter] Serializing blob to output stream...

________________________________________________________
Executed in    2.84 secs    fish           external
   usr time    2.07 secs  260.00 micros    2.07 secs
   sys time    0.30 secs  995.00 micros    0.30 secs

HEAD @ 2f5c923e0f0f06f5328c5c9addfd213767097a88

INFO  [icu_datagen] Writing key: segmenter/word@1
INFO  [icu_provider_blob::export::blob_exporter] Serializing blob to output stream...

________________________________________________________
Executed in    1.20 secs      fish           external
   usr time  968.95 millis  261.00 micros  968.68 millis
   sys time  440.98 millis  933.00 micros  440.05 millis

pdogr avatar May 06 '23 09:05 pdogr

Make sure the bench is writing multiple keys that all use the WASM module. My thesis is that there's a perf regression when the module is being reused.

sffc avatar May 07 '23 22:05 sffc

Well there's a perf regressions even for a single key...

robertbastian avatar May 08 '23 09:05 robertbastian

Discussion: we want to update to wasmer 3, but there shouldn't be a performance hit.

robertbastian avatar May 11 '23 17:05 robertbastian

The latest versions are now:

  • wasmer 4.1.1
  • wasmer-wasix 0.11.0

Perhaps the performance has been improved?

westy92 avatar Aug 05 '23 20:08 westy92

#4621

robertbastian avatar Feb 27 '24 17:02 robertbastian