diff --git a/Cargo.lock b/Cargo.lock index f402f5aef9..5948052629 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -550,6 +550,9 @@ dependencies = [ [[package]] name = "cranelift-codegen-shared" version = "0.70.0" +dependencies = [ + "serde", +] [[package]] name = "cranelift-entity" diff --git a/cranelift/codegen/Cargo.toml b/cranelift/codegen/Cargo.toml index 0bcdf471b4..5931434130 100644 --- a/cranelift/codegen/Cargo.toml +++ b/cranelift/codegen/Cargo.toml @@ -74,7 +74,12 @@ all-arch = [ ] # For dependent crates that want to serialize some parts of cranelift -enable-serde = ["serde", "regalloc/enable-serde"] +enable-serde = [ + "serde", + "regalloc/enable-serde", + "cranelift-entity/enable-serde", + "cranelift-codegen-shared/enable-serde" +] # Allow snapshotting regalloc test cases. Useful only to report bad register # allocation failures, or for regalloc.rs developers. diff --git a/cranelift/codegen/meta/src/gen_inst.rs b/cranelift/codegen/meta/src/gen_inst.rs index a2760b34d7..13f4776d3b 100644 --- a/cranelift/codegen/meta/src/gen_inst.rs +++ b/cranelift/codegen/meta/src/gen_inst.rs @@ -68,6 +68,7 @@ fn gen_formats(formats: &[&InstructionFormat], fmt: &mut Formatter) { /// `ValueList` to store the additional information out of line. fn gen_instruction_data(formats: &[&InstructionFormat], fmt: &mut Formatter) { fmt.line("#[derive(Clone, Debug)]"); + fmt.line(r#"#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))]"#); fmt.line("#[allow(missing_docs)]"); fmt.line("pub enum InstructionData {"); fmt.indent(|fmt| { @@ -410,7 +411,10 @@ fn gen_opcodes(all_inst: &AllInstructions, fmt: &mut Formatter) { fmt.line("#[repr(u16)]"); fmt.line("#[derive(Copy, Clone, PartialEq, Eq, Debug, Hash)]"); fmt.line( - r#"#[cfg_attr(feature = "enable-peepmatic", derive(serde::Serialize, serde::Deserialize))]"# + r#"#[cfg_attr( + any(feature = "enable-peepmatic", feature = "enable-serde"), + derive(serde::Serialize, serde::Deserialize) + )]"#, ); // We explicitly set the discriminant of the first variant to 1, which allows us to take diff --git a/cranelift/codegen/shared/Cargo.toml b/cranelift/codegen/shared/Cargo.toml index e25a528334..53f864e287 100644 --- a/cranelift/codegen/shared/Cargo.toml +++ b/cranelift/codegen/shared/Cargo.toml @@ -8,4 +8,10 @@ repository = "https://github.com/bytecodealliance/wasmtime" readme = "README.md" edition = "2018" -# Since this is a shared dependency of several packages, please strive to keep this dependency-free. +[dependencies] +# Since this is a shared dependency of several packages, please strive to keep this dependency-free +# when no features are enabled. +serde = { version = "1.0.94", features = ["derive"], optional = true } + +[features] +enable-serde = ["serde"] diff --git a/cranelift/codegen/shared/src/condcodes.rs b/cranelift/codegen/shared/src/condcodes.rs index 03ae865ce4..00e9717ca0 100644 --- a/cranelift/codegen/shared/src/condcodes.rs +++ b/cranelift/codegen/shared/src/condcodes.rs @@ -7,6 +7,9 @@ use core::fmt::{self, Display, Formatter}; use core::str::FromStr; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// Common traits of condition codes. pub trait CondCode: Copy { /// Get the inverse condition code of `self`. @@ -30,6 +33,7 @@ pub trait CondCode: Copy { /// separate codes for comparing the integers as signed or unsigned numbers where it makes a /// difference. #[derive(Clone, Copy, PartialEq, Eq, Debug, Hash)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub enum IntCC { /// `==`. Equal, @@ -187,6 +191,7 @@ impl FromStr for IntCC { /// comparison. The 14 condition codes here cover every possible combination of the relation above /// except the impossible `!UN & !EQ & !LT & !GT` and the always true `UN | EQ | LT | GT`. #[derive(Clone, Copy, PartialEq, Eq, Debug, Hash)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub enum FloatCC { /// EQ | LT | GT Ordered, diff --git a/cranelift/codegen/src/ir/constant.rs b/cranelift/codegen/src/ir/constant.rs index bd84e25ee3..a9aa5d3a64 100644 --- a/cranelift/codegen/src/ir/constant.rs +++ b/cranelift/codegen/src/ir/constant.rs @@ -19,12 +19,16 @@ use core::slice::Iter; use core::str::{from_utf8, FromStr}; use cranelift_entity::EntityRef; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// This type describes the actual constant data. Note that the bytes stored in this structure are /// expected to be in little-endian order; this is due to ease-of-use when interacting with /// WebAssembly values, which are [little-endian by design]. /// /// [little-endian by design]: https://github.com/WebAssembly/design/blob/master/Portability.md #[derive(Clone, Hash, Eq, PartialEq, Debug, Default)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct ConstantData(Vec); impl FromIterator for ConstantData { @@ -173,6 +177,7 @@ pub type ConstantOffset = u32; /// from the beginning of the function is known (see /// `relaxation` in `relaxation.rs`). #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct ConstantPoolEntry { data: ConstantData, offset: Option, @@ -197,6 +202,7 @@ impl ConstantPoolEntry { /// Maintains the mapping between a constant handle (i.e. [`Constant`](crate::ir::Constant)) and /// its constant data (i.e. [`ConstantData`](crate::ir::ConstantData)). #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct ConstantPool { /// This mapping maintains the insertion order as long as Constants are created with /// sequentially increasing integers. diff --git a/cranelift/codegen/src/ir/dfg.rs b/cranelift/codegen/src/ir/dfg.rs index 58d101aace..861f3147c5 100644 --- a/cranelift/codegen/src/ir/dfg.rs +++ b/cranelift/codegen/src/ir/dfg.rs @@ -21,6 +21,9 @@ use core::mem; use core::ops::{Index, IndexMut}; use core::u16; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// A data flow graph defines all instructions and basic blocks in a function as well as /// the data flow dependencies between them. The DFG also tracks values which can be either /// instruction results or block parameters. @@ -29,6 +32,7 @@ use core::u16; /// `Layout` data structure which forms the other half of the function representation. /// #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct DataFlowGraph { /// Data about all of the instructions in the function, including opcodes and operands. /// The instructions in this map are not in program order. That is tracked by `Layout`, along @@ -416,6 +420,7 @@ impl ValueDef { /// Internal table storage for extended values. #[derive(Clone, Debug)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] enum ValueData { /// Value is defined by an instruction. Inst { ty: Type, num: u16, inst: Inst }, @@ -935,6 +940,7 @@ impl DataFlowGraph { /// branches to this block must provide matching arguments, and the arguments to the entry block must /// match the function arguments. #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] struct BlockData { /// List of parameters to this block. params: ValueList, diff --git a/cranelift/codegen/src/ir/entities.rs b/cranelift/codegen/src/ir/entities.rs index e629475532..09eaed3bec 100644 --- a/cranelift/codegen/src/ir/entities.rs +++ b/cranelift/codegen/src/ir/entities.rs @@ -33,6 +33,7 @@ use serde::{Deserialize, Serialize}; /// /// While the order is stable, it is arbitrary and does not necessarily resemble the layout order. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Block(u32); entity_impl!(Block, "block"); @@ -65,6 +66,7 @@ impl Block { /// /// While the order is stable, it is arbitrary. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Value(u32); entity_impl!(Value, "v"); @@ -97,6 +99,7 @@ impl Value { /// /// While the order is stable, it is arbitrary and does not necessarily resemble the layout order. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Inst(u32); entity_impl!(Inst, "inst"); @@ -152,6 +155,7 @@ impl StackSlot { /// /// While the order is stable, it is arbitrary. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct GlobalValue(u32); entity_impl!(GlobalValue, "gv"); @@ -177,6 +181,7 @@ impl GlobalValue { /// While the order is stable, it is arbitrary and does not necessarily resemble the order in which /// the constants are written in the constant pool. #[derive(Copy, Clone, PartialEq, Eq, Hash, Ord, PartialOrd)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Constant(u32); entity_impl!(Constant, "const"); @@ -202,6 +207,7 @@ impl Constant { /// /// While the order is stable, it is arbitrary. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Immediate(u32); entity_impl!(Immediate, "imm"); @@ -267,6 +273,7 @@ impl JumpTable { /// /// While the order is stable, it is arbitrary. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct FuncRef(u32); entity_impl!(FuncRef, "fn"); @@ -298,6 +305,7 @@ impl FuncRef { /// /// While the order is stable, it is arbitrary. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct SigRef(u32); entity_impl!(SigRef, "sig"); @@ -323,6 +331,7 @@ impl SigRef { /// /// While the order is stable, it is arbitrary. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Heap(u32); entity_impl!(Heap, "heap"); @@ -349,6 +358,7 @@ impl Heap { /// /// While the order is stable, it is arbitrary. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Table(u32); entity_impl!(Table, "table"); @@ -367,6 +377,7 @@ impl Table { /// An opaque reference to any of the entities defined in this module that can appear in CLIF IR. #[derive(Copy, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub enum AnyEntity { /// The whole function. Function, diff --git a/cranelift/codegen/src/ir/extfunc.rs b/cranelift/codegen/src/ir/extfunc.rs index 46f1cf2828..6c1d26f4ab 100644 --- a/cranelift/codegen/src/ir/extfunc.rs +++ b/cranelift/codegen/src/ir/extfunc.rs @@ -409,6 +409,7 @@ impl FromStr for ArgumentPurpose { /// /// Information about a function that can be called directly with a direct `call` instruction. #[derive(Clone, Debug)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct ExtFuncData { /// Name of the external function. pub name: ExternalName, diff --git a/cranelift/codegen/src/ir/extname.rs b/cranelift/codegen/src/ir/extname.rs index c12a873d26..362cf8c67e 100644 --- a/cranelift/codegen/src/ir/extname.rs +++ b/cranelift/codegen/src/ir/extname.rs @@ -9,6 +9,9 @@ use core::cmp; use core::fmt::{self, Write}; use core::str::FromStr; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + const TESTCASE_NAME_LENGTH: usize = 16; /// The name of an external is either a reference to a user-defined symbol @@ -23,6 +26,7 @@ const TESTCASE_NAME_LENGTH: usize = 16; /// In particular, many `.clif` test files use function names to identify /// functions. #[derive(Debug, Clone, PartialEq, Eq)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub enum ExternalName { /// A name in a user-defined symbol table. Cranelift does not interpret /// these numbers in any way. diff --git a/cranelift/codegen/src/ir/function.rs b/cranelift/codegen/src/ir/function.rs index 1833af27f5..f5dc7f916f 100644 --- a/cranelift/codegen/src/ir/function.rs +++ b/cranelift/codegen/src/ir/function.rs @@ -21,11 +21,15 @@ use crate::write::write_function; use alloc::vec::Vec; use core::fmt; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// A function. /// /// Functions can be cloned, but it is not a very fast operation. /// The clone will have all the same entity numbers as the original. #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Function { /// Name of this function. Mostly used by `.clif` files. pub name: ExternalName, diff --git a/cranelift/codegen/src/ir/globalvalue.rs b/cranelift/codegen/src/ir/globalvalue.rs index 6c9b2d7bcf..e70f8221fb 100644 --- a/cranelift/codegen/src/ir/globalvalue.rs +++ b/cranelift/codegen/src/ir/globalvalue.rs @@ -6,8 +6,12 @@ use crate::isa::TargetIsa; use crate::machinst::RelocDistance; use core::fmt; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// Information about a global value declaration. #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub enum GlobalValueData { /// Value is the address of the VM context struct. VMContext, diff --git a/cranelift/codegen/src/ir/heap.rs b/cranelift/codegen/src/ir/heap.rs index 8a4b4e84b9..91aabccaa2 100644 --- a/cranelift/codegen/src/ir/heap.rs +++ b/cranelift/codegen/src/ir/heap.rs @@ -4,8 +4,12 @@ use crate::ir::immediates::Uimm64; use crate::ir::{GlobalValue, Type}; use core::fmt; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// Information about a heap declaration. #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct HeapData { /// The address of the start of the heap's storage. pub base: GlobalValue, @@ -26,6 +30,7 @@ pub struct HeapData { /// Style of heap including style-specific information. #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub enum HeapStyle { /// A dynamic heap can be relocated to a different base address when it is grown. Dynamic { diff --git a/cranelift/codegen/src/ir/immediates.rs b/cranelift/codegen/src/ir/immediates.rs index f575ea361f..9dd317d542 100644 --- a/cranelift/codegen/src/ir/immediates.rs +++ b/cranelift/codegen/src/ir/immediates.rs @@ -48,6 +48,7 @@ impl IntoBytes for Vec { /// An `Imm64` operand can also be used to represent immediate values of smaller integer types by /// sign-extending to `i64`. #[derive(Copy, Clone, PartialEq, Eq, Debug, Hash)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Imm64(i64); impl Imm64 { @@ -148,6 +149,7 @@ impl FromStr for Imm64 { /// A `Uimm64` operand can also be used to represent immediate values of smaller integer types by /// zero-extending to `i64`. #[derive(Copy, Clone, PartialEq, Eq, Debug, Hash)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Uimm64(u64); impl Uimm64 { @@ -279,6 +281,7 @@ pub type Uimm8 = u8; /// /// This is used to represent sizes of memory objects. #[derive(Copy, Clone, PartialEq, Eq, Debug, Hash)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Uimm32(u32); impl Into for Uimm32 { @@ -362,6 +365,7 @@ impl From<&[u8]> for V128Imm { /// This is used to encode an immediate offset for load/store instructions. All supported ISAs have /// a maximum load/store offset that fits in an `i32`. #[derive(Copy, Clone, PartialEq, Eq, Debug, Hash)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Offset32(i32); impl Offset32 { @@ -451,6 +455,7 @@ impl FromStr for Offset32 { /// /// All bit patterns are allowed. #[derive(Copy, Clone, Debug, Eq, PartialEq, Hash)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] #[repr(C)] pub struct Ieee32(u32); @@ -459,6 +464,7 @@ pub struct Ieee32(u32); /// /// All bit patterns are allowed. #[derive(Copy, Clone, Debug, Eq, PartialEq, Hash)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] #[repr(C)] pub struct Ieee64(u64); diff --git a/cranelift/codegen/src/ir/instructions.rs b/cranelift/codegen/src/ir/instructions.rs index 13310bc01c..94ce878a97 100644 --- a/cranelift/codegen/src/ir/instructions.rs +++ b/cranelift/codegen/src/ir/instructions.rs @@ -13,6 +13,9 @@ use core::num::NonZeroU32; use core::ops::{Deref, DerefMut}; use core::str::FromStr; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + use crate::ir::{self, trapcode::TrapCode, types, Block, FuncRef, JumpTable, SigRef, Type, Value}; use crate::isa; diff --git a/cranelift/codegen/src/ir/jumptable.rs b/cranelift/codegen/src/ir/jumptable.rs index a0596728a3..d102617df4 100644 --- a/cranelift/codegen/src/ir/jumptable.rs +++ b/cranelift/codegen/src/ir/jumptable.rs @@ -8,10 +8,14 @@ use alloc::vec::Vec; use core::fmt::{self, Display, Formatter}; use core::slice::{Iter, IterMut}; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// Contents of a jump table. /// /// All jump tables use 0-based indexing and are densely populated. #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct JumpTableData { // Table entries. table: Vec, diff --git a/cranelift/codegen/src/ir/layout.rs b/cranelift/codegen/src/ir/layout.rs index 62516325dd..9359c309df 100644 --- a/cranelift/codegen/src/ir/layout.rs +++ b/cranelift/codegen/src/ir/layout.rs @@ -781,6 +781,86 @@ impl<'f> DoubleEndedIterator for Insts<'f> { } } +#[cfg(feature = "enable-serde")] +mod serde { + use ::serde::de::{Deserializer, Error, SeqAccess, Visitor}; + use ::serde::ser::{SerializeSeq, Serializer}; + use ::serde::{Deserialize, Serialize}; + use core::convert::TryFrom; + use core::fmt; + use core::marker::PhantomData; + + use super::*; + + impl Serialize for Layout { + fn serialize(&self, serializer: S) -> Result + where + S: Serializer, + { + let size = self.blocks().count() * 2 + + self + .blocks() + .map(|block| self.block_insts(block).count()) + .sum::(); + let mut seq = serializer.serialize_seq(Some(size))?; + for block in self.blocks() { + seq.serialize_element(&block)?; + seq.serialize_element(&u32::try_from(self.block_insts(block).count()).unwrap())?; + for inst in self.block_insts(block) { + seq.serialize_element(&inst)?; + } + } + seq.end() + } + } + + impl<'de> Deserialize<'de> for Layout { + fn deserialize(deserializer: D) -> Result + where + D: Deserializer<'de>, + { + deserializer.deserialize_seq(LayoutVisitor { + marker: PhantomData, + }) + } + } + + struct LayoutVisitor { + marker: PhantomData Layout>, + } + + impl<'de> Visitor<'de> for LayoutVisitor { + type Value = Layout; + + fn expecting(&self, formatter: &mut fmt::Formatter) -> fmt::Result { + write!(formatter, "a `cranelift_codegen::ir::Layout`") + } + + fn visit_seq(self, mut access: M) -> Result + where + M: SeqAccess<'de>, + { + let mut layout = Layout::new(); + + while let Some(block) = access.next_element::()? { + layout.append_block(block); + + let count = access + .next_element::()? + .ok_or_else(|| Error::missing_field("count"))?; + for _ in 0..count { + let inst = access + .next_element::()? + .ok_or_else(|| Error::missing_field("inst"))?; + layout.append_inst(inst, block); + } + } + + Ok(layout) + } + } +} + #[cfg(test)] mod tests { use super::Layout; diff --git a/cranelift/codegen/src/ir/memflags.rs b/cranelift/codegen/src/ir/memflags.rs index 3c9c8c98ba..4ff76b623c 100644 --- a/cranelift/codegen/src/ir/memflags.rs +++ b/cranelift/codegen/src/ir/memflags.rs @@ -2,6 +2,9 @@ use core::fmt; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + enum FlagBit { Notrap, Aligned, @@ -32,6 +35,7 @@ pub enum Endianness { /// be overridden for individual accesses by explicitly specifying little- or big-endian /// semantics via the flags. #[derive(Clone, Copy, Debug, Hash, PartialEq, Eq)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct MemFlags { bits: u8, } diff --git a/cranelift/codegen/src/ir/mod.rs b/cranelift/codegen/src/ir/mod.rs index c78dde81de..c075da6824 100644 --- a/cranelift/codegen/src/ir/mod.rs +++ b/cranelift/codegen/src/ir/mod.rs @@ -91,6 +91,7 @@ entity_impl!(ValueLabel, "val"); /// A label of a Value. #[derive(Debug, Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct ValueLabelStart { /// Source location when it is in effect pub from: SourceLoc, @@ -101,6 +102,7 @@ pub struct ValueLabelStart { /// Value label assignements: label starts or value aliases. #[derive(Debug, Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub enum ValueLabelAssignments { /// Original value labels assigned at transform. Starts(alloc::vec::Vec), diff --git a/cranelift/codegen/src/ir/table.rs b/cranelift/codegen/src/ir/table.rs index 9e436cca64..713d1f5df7 100644 --- a/cranelift/codegen/src/ir/table.rs +++ b/cranelift/codegen/src/ir/table.rs @@ -4,8 +4,12 @@ use crate::ir::immediates::Uimm64; use crate::ir::{GlobalValue, Type}; use core::fmt; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// Information about a table declaration. #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct TableData { /// Global value giving the address of the start of the table. pub base_gv: GlobalValue, diff --git a/cranelift/codegen/src/isa/encoding.rs b/cranelift/codegen/src/isa/encoding.rs index 99894cab2c..84001c5d36 100644 --- a/cranelift/codegen/src/isa/encoding.rs +++ b/cranelift/codegen/src/isa/encoding.rs @@ -6,6 +6,9 @@ use crate::isa::constraints::{BranchRange, RecipeConstraints}; use crate::regalloc::RegDiversions; use core::fmt; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// Bits needed to encode an instruction as binary machine code. /// /// The encoding consists of two parts, both specific to the target ISA: An encoding *recipe*, and @@ -13,6 +16,7 @@ use core::fmt; /// operands to encoded bits. The encoding bits provide additional information to the recipe, /// typically parts of the opcode. #[derive(Clone, Copy, Debug, PartialEq, Eq)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Encoding { recipe: u16, bits: u16, diff --git a/cranelift/codegen/src/regalloc/diversion.rs b/cranelift/codegen/src/regalloc/diversion.rs index be7b7fa1ee..ba91df14a5 100644 --- a/cranelift/codegen/src/regalloc/diversion.rs +++ b/cranelift/codegen/src/regalloc/diversion.rs @@ -15,6 +15,9 @@ use crate::isa::{RegInfo, RegUnit}; use core::fmt; use cranelift_entity::{SparseMap, SparseMapValue}; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// A diversion of a value from its original location to a new register or stack location. /// /// In IR, a diversion is represented by a `regmove` instruction, possibly a chain of them for the @@ -23,6 +26,7 @@ use cranelift_entity::{SparseMap, SparseMapValue}; /// When tracking diversions, the `from` field is the original assigned value location, and `to` is /// the current one. #[derive(Clone, Copy, Debug, PartialEq, Eq)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct Diversion { /// The original value location. pub from: ValueLoc, @@ -40,18 +44,21 @@ impl Diversion { /// Keep track of diversions in a block. #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct RegDiversions { current: FxHashMap, } /// Keep track of diversions at the entry of block. #[derive(Clone)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] struct EntryRegDiversionsValue { key: Block, divert: RegDiversions, } /// Map block to their matching RegDiversions at basic blocks entry. +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct EntryRegDiversions { map: SparseMap, } diff --git a/cranelift/entity/src/list.rs b/cranelift/entity/src/list.rs index dfb8fac646..3e583c5fd7 100644 --- a/cranelift/entity/src/list.rs +++ b/cranelift/entity/src/list.rs @@ -5,6 +5,9 @@ use alloc::vec::Vec; use core::marker::PhantomData; use core::mem; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// A small list of entity references allocated from a pool. /// /// An `EntityList` type provides similar functionality to `Vec`, but with some important @@ -59,7 +62,8 @@ use core::mem; /// /// The index stored in an `EntityList` points to part 2, the list elements. The value 0 is /// reserved for the empty list which isn't allocated in the vector. -#[derive(Clone, Debug)] +#[derive(Clone, Debug, PartialEq)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct EntityList { index: u32, unused: PhantomData, @@ -77,6 +81,7 @@ impl Default for EntityList { /// A memory pool for storing lists of `T`. #[derive(Clone, Debug)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct ListPool { // The main array containing the lists. data: Vec, diff --git a/cranelift/entity/src/packed_option.rs b/cranelift/entity/src/packed_option.rs index 3bca32f8a0..7f48b0196a 100644 --- a/cranelift/entity/src/packed_option.rs +++ b/cranelift/entity/src/packed_option.rs @@ -10,6 +10,9 @@ use core::fmt; use core::mem; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// Types that have a reserved value which can't be created any other way. pub trait ReservedValue { /// Create an instance of the reserved value. @@ -20,6 +23,7 @@ pub trait ReservedValue { /// Packed representation of `Option`. #[derive(Clone, Copy, PartialEq, PartialOrd, Eq, Ord, Hash)] +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct PackedOption(T); impl PackedOption { diff --git a/cranelift/entity/src/sparse.rs b/cranelift/entity/src/sparse.rs index 57d971b281..a7cad3151c 100644 --- a/cranelift/entity/src/sparse.rs +++ b/cranelift/entity/src/sparse.rs @@ -14,6 +14,9 @@ use core::mem; use core::slice; use core::u32; +#[cfg(feature = "enable-serde")] +use serde::{Deserialize, Serialize}; + /// Trait for extracting keys from values stored in a `SparseMap`. /// /// All values stored in a `SparseMap` must keep track of their own key in the map and implement @@ -53,6 +56,7 @@ pub trait SparseMapValue { /// is). /// - `SparseMap` requires the values to implement `SparseMapValue` which means that they must /// contain their own key. +#[cfg_attr(feature = "enable-serde", derive(Serialize, Deserialize))] pub struct SparseMap where K: EntityRef,