Merge remote-tracking branch 'private/feature/dynasm-backend' into feature/dynasm-backend

This commit is contained in:
losfair 2019-03-17 03:08:06 +08:00
commit 1f8c644855
3 changed files with 8 additions and 5 deletions

View File

@ -48,7 +48,7 @@ test-emscripten:
release:
# If you are in OS-X, you will need mingw-w64 for cross compiling to windows
# brew install mingw-w64
cargo build --release
cargo +nightly build --release
debug-release:
cargo build --release --features debug

View File

@ -38,7 +38,8 @@ impl CacheGen for Placeholder {
&self,
module: &ModuleInner,
) -> Result<(Box<ModuleInfo>, Box<[u8]>, Memory), CacheError> {
unimplemented!()
// unimplemented!()
Err(CacheError::Unknown("the dynasm backend doesn't support caching yet".to_string()))
}
}
@ -73,7 +74,8 @@ impl Compiler for SinglePassCompiler {
}
unsafe fn from_cache(&self, _artifact: Artifact, _: Token) -> Result<ModuleInner, CacheError> {
unimplemented!("the dynasm backend doesn't support caching yet")
Err(CacheError::Unknown("the dynasm backend doesn't support caching yet".to_string()))
// unimplemented!("the dynasm backend doesn't support caching yet")
}
}

View File

@ -128,8 +128,9 @@ fn execute_wasm(options: &Run) -> Result<(), String> {
let module = webassembly::compile(&wasm_binary[..])
.map_err(|e| format!("Can't compile module: {:?}", e))?;
// We save the module into a cache file
cache.store(hash, module.clone()).unwrap();
// We try to save the module into a cache file
cache.store(hash, module.clone()).unwrap_or_default();
module
}
};