T0b1/wasmtime
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix pretty print of atomic_rmw clif ops (#4441)

Browse Source 
A comma was accidentally placed after the operation itself when no comma
is necessary (only after operands, not the operation itself).
This commit is contained in:
Alex Crichton
2022-07-13 10:51:07 -05:00
committed by GitHub
parent e31ff9dc67
commit 18d9685eb3
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cranelift/codegen/src/write.rs
View File

@@ -380,7 +380,7 @@ pub fn write_operands(w: &mut dyn Write, dfg: &DataFlowGraph, inst: Inst) -> fmt
let pool = &dfg.value_lists; let pool = &dfg.value_lists;
use crate::ir::instructions::InstructionData::*; use crate::ir::instructions::InstructionData::*;
match dfg[inst] { match dfg[inst] {
AtomicRmw { op, args, .. } => write!(w, " {}, {}, {}", op, args[0], args[1]), AtomicRmw { op, args, .. } => write!(w, " {} {}, {}", op, args[0], args[1]),
AtomicCas { args, .. } => write!(w, " {}, {}, {}", args[0], args[1], args[2]), AtomicCas { args, .. } => write!(w, " {}, {}, {}", args[0], args[1], args[2]),
LoadNoOffset { flags, arg, .. } => write!(w, "{} {}", flags, arg), LoadNoOffset { flags, arg, .. } => write!(w, "{} {}", flags, arg),
StoreNoOffset { flags, args, .. } => write!(w, "{} {}, {}", flags, args[0], args[1]), StoreNoOffset { flags, args, .. } => write!(w, "{} {}, {}", flags, args[0], args[1]),