-
-
Notifications
You must be signed in to change notification settings - Fork 14.9k
cfg(wasm) unifying target_arch="wasm32" and target_arch="wasm64" #83879
Copy link
Copy link
Closed
Labels
O-wasmTarget: WASM (WebAssembly), http://webassembly.org/Target: WASM (WebAssembly), http://webassembly.org/T-langRelevant to the language teamRelevant to the language team
Description
Metadata
Metadata
Assignees
Labels
O-wasmTarget: WASM (WebAssembly), http://webassembly.org/Target: WASM (WebAssembly), http://webassembly.org/T-langRelevant to the language teamRelevant to the language team
Type
Fields
Give feedbackNo fields configured for issues without a type.
In #80525 we added to nightly a
cfg(wasm)which is set when either wasm32 or wasm64 targets are used. We don't have top-level cfg values for other architectures in a similar situation, only for windows vs unix which is set based on the OS.This ended up being an insta-stable addition due to oversight of mine.
This issue is in order to gather consensus from T-lang members that we do indeed want this
cfg. I will make myself a note to either remove thiscfgor put it behind a feature gate before the next nightly->beta rollover that happens on May 6th if such consensus is not gathered.Zulip discussion
PR adding the cfg