diff --git a/cranelift/module/src/module.rs b/cranelift/module/src/module.rs
index f27cac2012..4e08afe37e 100644
--- a/cranelift/module/src/module.rs
+++ b/cranelift/module/src/module.rs
@@ -191,6 +191,16 @@ where
         }
         Ok(())
     }
+
+    fn validate_for_define(&self) -> ModuleResult<()> {
+        if self.compiled.is_some() {
+            return Err(ModuleError::DuplicateDefinition(self.decl.name.clone()));
+        }
+        if !self.decl.linkage.is_definable() {
+            return Err(ModuleError::InvalidImportDefinition(self.decl.name.clone()));
+        }
+        Ok(())
+    }
 }
 
 /// Information about a data object which can be accessed.
@@ -226,6 +236,16 @@ where
             "Can't change TLS data object to normal or in the opposite way",
         );
     }
+
+    fn validate_for_define(&self) -> ModuleResult<()> {
+        if self.compiled.is_some() {
+            return Err(ModuleError::DuplicateDefinition(self.decl.name.clone()));
+        }
+        if !self.decl.linkage.is_definable() {
+            return Err(ModuleError::InvalidImportDefinition(self.decl.name.clone()));
+        }
+        Ok(())
+    }
 }
 
 /// This provides a view to the state of a module which allows `ir::ExternalName`s to be translated
@@ -553,12 +573,7 @@ where
         );
         let CodeInfo { total_size, .. } = ctx.compile(self.backend.isa())?;
         let info = &self.contents.functions[func];
-        if info.compiled.is_some() {
-            return Err(ModuleError::DuplicateDefinition(info.decl.name.clone()));
-        }
-        if !info.decl.linkage.is_definable() {
-            return Err(ModuleError::InvalidImportDefinition(info.decl.name.clone()));
-        }
+        info.validate_for_define()?;
 
         let compiled = self.backend.define_function(
             func,
@@ -588,12 +603,7 @@ where
     ) -> ModuleResult<ModuleCompiledFunction> {
         info!("defining function {} with bytes", func);
         let info = &self.contents.functions[func];
-        if info.compiled.is_some() {
-            return Err(ModuleError::DuplicateDefinition(info.decl.name.clone()));
-        }
-        if !info.decl.linkage.is_definable() {
-            return Err(ModuleError::InvalidImportDefinition(info.decl.name.clone()));
-        }
+        info.validate_for_define()?;
 
         let total_size: u32 = match bytes.len().try_into() {
             Ok(total_size) => total_size,
@@ -613,12 +623,7 @@ where
     pub fn define_data(&mut self, data: DataId, data_ctx: &DataContext) -> ModuleResult<()> {
         let compiled = {
             let info = &self.contents.data_objects[data];
-            if info.compiled.is_some() {
-                return Err(ModuleError::DuplicateDefinition(info.decl.name.clone()));
-            }
-            if !info.decl.linkage.is_definable() {
-                return Err(ModuleError::InvalidImportDefinition(info.decl.name.clone()));
-            }
+            info.validate_for_define()?;
             Some(self.backend.define_data(
                 data,
                 &info.decl.name,