Skip to content

Commit 0891f25

Browse files
alexcrichtonabrown
andauthored
[30.0.0] asm: avoid propagating arbitrary (#10295)
* asm: avoid propagating `arbitrary` (#10217) This is something that @alexcrichton already fixed over in the assembler crate--no need to implement `Arbitrary` unnecessarily. It's unclear how this stayed in here in `cranelift-codegen` but this change removes it. * Update release notes --------- Co-authored-by: Andrew Brown <[email protected]>
1 parent 3f9bf16 commit 0891f25

File tree

4 files changed

+5
-15
lines changed

4 files changed

+5
-15
lines changed

Cargo.lock

Lines changed: 0 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

RELEASES.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,11 @@ Released 2025-02-25.
88
fix an issue caused by #9929.
99
[#10290](https://github.com/bytecodealliance/wasmtime/pull/10290)
1010

11+
### Changed
12+
13+
* The `cranelift-codegen` crate now no longer depends on `arbitrary`, a
14+
now-unnecessary dependency.
15+
[#10217](https://github.com/bytecodealliance/wasmtime/pull/10217)
1116

1217
--------------------------------------------------------------------------------
1318

cranelift/codegen/Cargo.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ workspace = true
2121
features = ["all-arch"]
2222

2323
[dependencies]
24-
arbitrary = { version = "1.3.2", features = ["derive"] }
2524
anyhow = { workspace = true, optional = true, features = ['std'] }
2625
bumpalo = "3"
2726
capstone = { workspace = true, optional = true }

cranelift/codegen/src/isa/x64/inst/external.rs

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ use super::{
55
SyntheticAmode, VCodeConstant, WritableGpr,
66
};
77
use crate::ir::TrapCode;
8-
use arbitrary::Arbitrary;
98
use cranelift_assembler_x64 as asm;
109

1110
/// Define the types of registers Cranelift will use.
@@ -42,12 +41,6 @@ impl asm::AsReg for PairedGpr {
4241
}
4342
}
4443

45-
impl<'a> Arbitrary<'a> for PairedGpr {
46-
fn arbitrary(_: &mut arbitrary::Unstructured<'a>) -> arbitrary::Result<Self> {
47-
unimplemented!("assembler fuzzing is not implemented at this level")
48-
}
49-
}
50-
5144
/// This bridges the gap between codegen and assembler register types.
5245
impl asm::AsReg for Gpr {
5346
fn enc(&self) -> u8 {
@@ -59,12 +52,6 @@ impl asm::AsReg for Gpr {
5952
}
6053
}
6154

62-
impl<'a> Arbitrary<'a> for Gpr {
63-
fn arbitrary(_: &mut arbitrary::Unstructured<'a>) -> arbitrary::Result<Self> {
64-
unimplemented!("assembler fuzzing is not implemented at this level")
65-
}
66-
}
67-
6855
/// A helper method for extracting the hardware encoding of a register.
6956
#[inline]
7057
fn enc(gpr: &Gpr) -> u8 {

0 commit comments

Comments
 (0)