Skip to content

Commit 8c834e8

Browse files
authored
When creating dynCall and legalstub function mark them as hasExplicitName. NFC (#6496)
This is temporary workaround for the current test failures on the emscripten waterfall. The real fix I believe will be to make `hasExplicitName` the default in these kinds of cases. See: #6466
1 parent 111902d commit 8c834e8

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

src/passes/GenerateDynCalls.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,7 @@ void GenerateDynCalls::generateDynCallThunk(HeapType funcType) {
149149
}
150150
auto f = builder.makeFunction(
151151
name, std::move(namedParams), Signature(Type(params), sig.results), {});
152+
f->hasExplicitName = true;
152153
Expression* fptr = builder.makeLocalGet(0, Type::i32);
153154
std::vector<Expression*> args;
154155
Index i = 0;

src/passes/LegalizeJSInterface.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,7 @@ struct LegalizeJSInterface : public Pass {
229229
Builder builder(*module);
230230
auto* legal = new Function();
231231
legal->name = legalName;
232+
legal->hasExplicitName = true;
232233

233234
auto* call = module->allocator.alloc<Call>();
234235
call->target = func->name;

0 commit comments

Comments
 (0)