Skip to content

Commit

Permalink
Don't generate 128-bit atomics in fuzzing
Browse files Browse the repository at this point in the history
This commit fixes some issues cropping up due to fuzzing 128-bit atomics
in `cranelift-{icache,fuzzgen}`. Not all backends support these
operations and x64 only supports them with a feature enabled, so for now
disable fuzzing until it's ready to fuzz later.
  • Loading branch information
alexcrichton committed Oct 16, 2024
1 parent 3f2febf commit a7bb508
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions cranelift/fuzzgen/src/function_generator.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1051,6 +1051,12 @@ static OPCODE_SIGNATURES: Lazy<Vec<OpcodeSignature>> = Lazy::new(|| {
(Opcode::FcvtFromSint, &[I8X16], &[F64X2]),
(Opcode::FcvtFromSint, &[I16X8], &[F64X2]),
(Opcode::FcvtFromSint, &[I32X4], &[F64X2]),
// Only supported on x64 with a feature at this time, so 128-bit
// atomics are not suitable to fuzz yet.
(Opcode::AtomicRmw, _, &[I128]),
(Opcode::AtomicCas, _, &[I128]),
(Opcode::AtomicLoad, _, &[I128]),
(Opcode::AtomicStore, &[I128, _], _),
)
})
.filter(|(op, ..)| {
Expand Down

0 comments on commit a7bb508

Please sign in to comment.