...
This can be seen my looking at emitLoadOp. This is part of the reason we can't unity. However, there are still other errors I'm hitting after fixing this locally.
<rdar://problem/29841541>
Created attachment 297906 [details] patch
Comment on attachment 297906 [details] patch lgtm
Forgot to ask: are there new spec tests that pass?
(In reply to comment #5) > Forgot to ask: are there new spec tests that pass? I didn't try all of them, but memory.wast.js still fails. I can check all of them.
Created attachment 297914 [details] patch Two more tests pass for me locally, but I'm not convinced they're related to this patch. They could be tests we forgot to update.
Comment on attachment 297914 [details] patch r=me if you add a test for each load.
landed in: https://trac.webkit.org/changeset/210228 with tests for all loads that were missing offset.