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.

Sign up for GitHub

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

Jump to bottom

change func_index offset by number of imported funcs #325

Merged
merged 1 commit into from
Apr 4, 2019
Merged

change func_index offset by number of imported funcs #325

merged 1 commit into from
Apr 4, 2019

Conversation

MarkMcCaskey
Copy link
Contributor

This appears to fix the issue of the debug statements reporting the wrong function index

@MarkMcCaskey MarkMcCaskey added the 📦 lib-emscripten About wasmer-emscripten label Apr 4, 2019
@MarkMcCaskey MarkMcCaskey requested a review from syrusakbary April 4, 2019 16:44
@syrusakbary
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request Apr 4, 2019
@bors
Merge #325
e03a4a2 
325: change func_index offset by number of imported funcs r=syrusakbary a=MarkMcCaskey

This appears to fix the issue of the debug statements reporting the wrong function index

Co-authored-by: Mark McCaskey <mark@wasmer.io>
@bors
Copy link
Contributor

bors bot commented Apr 4, 2019

Build succeeded

@bors bors bot merged commit 9e601c4 into master Apr 4, 2019
@bors bors bot deleted the fix/func-index-offset branch April 4, 2019 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@syrusakbary syrusakbary syrusakbary approved these changes

Assignees
No one assigned
Labels
📦 lib-emscripten About wasmer-emscripten
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MarkMcCaskey @syrusakbary