Skip to content

I64ToI32Lowering: don't use alignment 1 everywhere#2070

Merged
kripken merged 6 commits intomasterfrom
wasm2js5
Apr 30, 2019
Merged

I64ToI32Lowering: don't use alignment 1 everywhere#2070
kripken merged 6 commits intomasterfrom
wasm2js5

Conversation

@kripken
Copy link
Copy Markdown
Member

@kripken kripken commented Apr 30, 2019

If an i64 load/store that is being broken up has higher alignment, use that.

@kripken kripken merged commit 06f659c into master Apr 30, 2019
@kripken kripken deleted the wasm2js5 branch April 30, 2019 19:29
@tlively
Copy link
Copy Markdown
Member

tlively commented Apr 30, 2019

Should there be a more conservative mode where single-byte alignment is used everywhere to guard against incorrect alignment hints?

@kripken
Copy link
Copy Markdown
Member Author

kripken commented Apr 30, 2019

I forgot earlier, but SAFE_HEAP should do that, it throws on incorrect alignment, and not just reads of 0 or beyond the memory size.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants