Skip to content

[wasm] Disable llvm for methods/calls with more than 1000 parameters. - #80243

Merged
vargaz merged 1 commit into
dotnet:mainfrom
vargaz:wasm-param-count
Jan 5, 2023
Merged

[wasm] Disable llvm for methods/calls with more than 1000 parameters.#80243
vargaz merged 1 commit into
dotnet:mainfrom
vargaz:wasm-param-count

Conversation

@vargaz

@vargazvargaz commented Jan 5, 2023

Copy link
Copy Markdown
Contributor

Some test suites are using methods like it and it causes a load error in some v8 versions.

@vargaz
[wasm] Disable llvm for methods/calls with more than 1000 parameters.
6fe3dd7
Some test suites are using methods like it and it causes a load
error in some v8 versions.
@ghostghost assigned vargazJan 5, 2023
@vargaz

vargaz commented Jan 5, 2023

Copy link
Copy Markdown
ContributorAuthor

/azp run runtime-wasm

@azure-pipelines

azure-pipelinesBot commented Jan 5, 2023

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@build-analysisbuild-analysisBot mentioned this pull request Jan 5, 2023
Closed
@vargaz
vargaz merged commit 1801b56 into dotnet:mainJan 5, 2023
@vargaz
vargaz deleted the wasm-param-count branch January 5, 2023 23:35
@ghostghost locked as resolved and limited conversation to collaborators Feb 5, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@lewinglewinglewing approved these changes

Assignees

@vargazvargaz

Labels

area-Codegen-LLVM-mono

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vargaz@lewing