chinhkrb113 opened PR #13517 from chinhkrb113:improve/quality/re-export-jitmemorykind-from-the-crate-r to bytecodealliance:main:
Summary
Making
JITMemoryKindpublic inmemory.rsis not sufficient if it is not re-exported from the crate root alongsideJITMemoryProviderand the other memory-related public items. Downstream users import these symbols fromcranelift_jit::..., so the type needs to be reachable through the same path used forJITMemoryProviderFixes #13200
Changes
cranelift/jit/src/lib.rsWhy
cranelift/jit/src/lib.rs: MakingJITMemoryKindpublic inmemory.rsis not sufficient if it is not re-exported from the crate root alongsideJITMemoryProviderand the other memory-related public items. Downstream users import these symbols fromcranelift_jit::..., so the type needs to be reachable through the same path used forJITMemoryProvider.
chinhkrb113 requested wasmtime-compiler-reviewers for a review on PR #13517.
chinhkrb113 requested fitzgen for a review on PR #13517.
github-actions[bot] added the label cranelift on PR #13517.
fitzgen commented on PR #13517:
Thanks, but it looks like you didn't run tests because there are compile errors:
Compiling cranelift-jit v0.133.0 (/home/runner/work/wasmtime/wasmtime/cranelift/jit) error: missing documentation for an enum --> cranelift/jit/src/memory/mod.rs:19:1 | 19 | pub enum JITMemoryKind { | ^^^^^^^^^^^^^^^^^^^^^^ | note: the lint level is defined here --> cranelift/jit/src/lib.rs:6:9 | 6 | #![deny(missing_docs, unreachable_pub)] | ^^^^^^^^^^^^ error: could not compile `cranelift-jit` (lib) due to 1 previous error
:cross_mark: alexcrichton closed without merge PR #13517.
alexcrichton commented on PR #13517:
Landing in https://github.com/bytecodealliance/wasmtime/pull/13632
Last updated: Jul 29 2026 at 05:03 UTC