Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
dynCallLegacy: Fill in dynCall_sig with a Wasm adaptor if it is missing #17328
dynCallLegacy: Fill in dynCall_sig with a Wasm adaptor if it is missing #17328
Changes from 7 commits
9121505
380052c
b678b94
99b057d
ffc9857
905e4c5
e1e0e0a
e90c349
2a2bb84
ab67a52
14a3c1a
1341807
5cde82b
0ff7b96
fd40684
8a477cb
2b95712
fe19443
49ab4c2
1950321
d989126
c32bf3b
bc55996
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really huge and unpleasant function.
I get that we might need it under certain circumstances but maybe we can take some steps to limits its impact. How about moving it to its own file? Maybe:
src/library_makeDynCall.js
Is there some way we can have folks opt into it? I'm loath to add a new setting. Perhaps we could ask folks to do
-lmakeDynCall
or--js-library=makeDynCall
? (Let me see if either of those will work).