Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alignment/Memory access problem for the wasm32-wasi target #68885

Closed
lu-zero opened this issue Feb 6, 2020 · 1 comment
Closed

Alignment/Memory access problem for the wasm32-wasi target #68885

lu-zero opened this issue Feb 6, 2020 · 1 comment
Labels
A-codegen Area: Code generation C-bug Category: This is a bug. E-needs-mcve Call for participation: This issue has a repro, but needs a Minimal Complete and Verifiable Example O-wasm Target: WASM (WebAssembly), http://webassembly.org/ T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@lu-zero
Copy link
Contributor

lu-zero commented Feb 6, 2020

here a full testcase.

Any alignment over 64bit seems to cause the problem even I would expect v128 to require 128bit alignment.

@jonas-schievink jonas-schievink added A-codegen Area: Code generation C-bug Category: This is a bug. O-wasm Target: WASM (WebAssembly), http://webassembly.org/ T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 6, 2020
@nagisa nagisa added the E-needs-mcve Call for participation: This issue has a repro, but needs a Minimal Complete and Verifiable Example label Feb 9, 2020
@alexcrichton
Copy link
Member

I believe this was fixed by #70585, so closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-codegen Area: Code generation C-bug Category: This is a bug. E-needs-mcve Call for participation: This issue has a repro, but needs a Minimal Complete and Verifiable Example O-wasm Target: WASM (WebAssembly), http://webassembly.org/ T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

4 participants