It implies to create the `wasm::Memory` trait.
Also, the patch updates `wasm::Type` and `wasm::Value` to
`wasm::InterfaceType` and `wasm::InterfaceValue`. It enforces a new
rule that is: All values in the stack must be owned by the stack. Any
value going in or out must be cloned.
802: This is no longer referring to the correct line. r=nlewycky a=nlewycky
It's supposed to be 352 which is already there. Three lines were deleted in #785.
Co-authored-by: Nick Lewycky <nick@wasmer.io>
Depending on the micro-arch, this function may trigger emission of a stack realignment. If it does, OSR can't statically prove that it's safe to insert a function call and asserts.
790: Switch llvm-backend from PIC+Default (small) to Static+Large. r=nlewycky a=nlewycky
# Description
Should fix flaky test failure:
```
spectest-a277b7d051ac0dde: /home/nick/llvm-8.0.1.src/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp:304: void llvm::RuntimeDyldELF::resolveX86_64Relocation(const llvm::SectionEntry&, uint64_t, uint64_t, uint32_t, int64_t, uint64_t): Assertion `isInt<32>(RealOffset)' failed.
```
(with assertions-enabled LLVM). The relocation in question is a `R_X86_64_PC32` with small offset against `.rodata`.
The switch from Default (small, JITDefault is a different default that we weren't using which is medium) to Large should fix the bug. Once we're using large code model, I think we can switch from PIC to static as either an optimization or a no-op.
# Review
- [x] Create a short description of the the change in the CHANGELOG.md file
Co-authored-by: Nick Lewycky <nick@wasmer.io>
798: Replace panics in codegen_x64 by CodegenError r=nlewycky a=pventuzelo
Replace panics! inside `lib/singlepass-backend/src/codegen_x64.rs` by CodegenError
Co-authored-by: Patrick Ventuzelo <ventuzelo.patrick@gmail.com>
782: Turn this unreachable into a panic with an error message. r=nlewycky a=nlewycky
# Description
All errors with messages in this file are `panic!`s. Give this `unreachable!` a message (so I can debug when I hit it, only possible due to local changes) and turn it into a `panic!`.
# Review
- [ ] Create a short description of the the change in the CHANGELOG.md file
Too minor for changelog.
Co-authored-by: Nick Lewycky <nick@wasmer.io>
Co-authored-by: nlewycky <nick@wasmer.io>
788: Use union merge on the changelog file. r=MarkMcCaskey a=nlewycky
# Description
This keeps both sides of a conflict. It's essentially the same as removing the
conflict markers.
# Review
- [x] Create a short description of the the change in the CHANGELOG.md file
Co-authored-by: Nick Lewycky <nick@wasmer.io>
