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

enable wasm non-trapping float to int conversions #5129

Merged
merged 1 commit into from
May 11, 2018
Merged

enable wasm non-trapping float to int conversions #5129

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion lib/Common/ConfigFlagsList.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -426,6 +426,7 @@ PHASE(All)
#define DEFAULT_CONFIG_WasmMaxTableSize (10000000)
#define DEFAULT_CONFIG_WasmThreads (false)
#define DEFAULT_CONFIG_WasmMultiValue (false)
#define DEFAULT_CONFIG_WasmNontrapping (true)
#define DEFAULT_CONFIG_WasmExperimental (false)
#define DEFAULT_CONFIG_BgJitDelayFgBuffer (0)
#define DEFAULT_CONFIG_BgJitPendingFuncCap (31)
Expand Down Expand Up @@ -933,6 +934,7 @@ FLAGNR(Boolean, WasmIgnoreResponse , "Ignore the type of the Response object"
FLAGNR(Number, WasmMaxTableSize , "Maximum size allowed to the WebAssembly.Table", DEFAULT_CONFIG_WasmMaxTableSize)
FLAGNR(Boolean, WasmThreads , "Enable WebAssembly threads feature", DEFAULT_CONFIG_WasmThreads)
FLAGNR(Boolean, WasmMultiValue , "Use new WebAssembly multi-value", DEFAULT_CONFIG_WasmMultiValue)
FLAGNR(Boolean, WasmNontrapping, "Enable non-trapping float-to-int conversions in WebAssembly", DEFAULT_CONFIG_WasmNontrapping)

// WebAssembly Experimental Features
// Master WasmExperimental flag to activate WebAssembly experimental features
Expand All @@ -944,7 +946,6 @@ FLAGR(Boolean, WasmExperimental, "Enable WebAssembly experimental features", DEF
// Not having the DEFAULT_CONFIG_XXXX macro ensures we use CONFIG_FLAG_RELEASE instead of CONFIG_FLAG
FLAGPR_EXPERIMENTAL_WASM(Boolean, WasmSignExtends , "Use new WebAssembly sign extension operators")
FLAGPR_EXPERIMENTAL_WASM(Boolean, WasmSimd , "Enable SIMD in WebAssembly")
FLAGPR_EXPERIMENTAL_WASM(Boolean, WasmNontrapping, "Enable non-trapping float-to-int conversions in WebAssembly")

FLAGNR(Boolean, AssertBreak , "Debug break on assert", false)
FLAGNR(Boolean, AssertPopUp , "Pop up asserts (default: false)", false)
Expand Down
2 changes: 1 addition & 1 deletion lib/WasmReader/WasmParseTree.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ namespace WasmNontrapping
bool IsEnabled()
{
#ifdef ENABLE_WASM
return CONFIG_FLAG_RELEASE(WasmNontrapping);
return CONFIG_FLAG(WasmNontrapping);
#else
return false;
#endif
Expand Down