Closed
Bug 1739608
Opened 3 years ago
Closed 3 years ago
Permanent spidermonkey non-unified bustage js/src/wasm/WasmBCMemory.cpp:813:24: error: unused function 'ToRegister' [-Werror,-Wunused-function] when Gecko 96 merges to Beta on 2021-12-06
Categories
(Core :: JavaScript: WebAssembly, defect, P2)
Core
JavaScript: WebAssembly
Tracking
()
VERIFIED
FIXED
96 Branch
Tracking | Status | |
---|---|---|
firefox-esr91 | --- | unaffected |
firefox93 | --- | unaffected |
firefox94 | --- | unaffected |
firefox95 | --- | fixed |
firefox96 | + | verified |
People
(Reporter: aryx, Assigned: lth)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
- central-as-beta simulation with fix for bug 1739603 applied (not needed to reproduce this failure but other spidermonkey and Firefox builds are busted without)
- How to run these simulations
- Failure log
js/src/wasm/WasmBCMemory.cpp:813:24: error: unused function 'ToRegister' [-Werror,-Wunused-function]
Flags: needinfo?(lhansen)
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•3 years ago
|
Assignee: nobody → lhansen
Status: NEW → ASSIGNED
Flags: needinfo?(lhansen)
Priority: -- → P2
Assignee | ||
Comment 2•3 years ago
|
||
Note, nonunified builds only.
Assignee | ||
Comment 3•3 years ago
|
||
Ifdef the ToRegister(RegI64) function on the memory64 feature, as it
is only used with that feature.
Pushed by lhansen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b7784534c25f Hide a function when appropriate. r=jseward
Updated•3 years ago
|
Keywords: regression
Updated•3 years ago
|
Comment 5•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
Comment 6•3 years ago
•
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/eee9e3d67c84
(uplifted becaused bug 1738997 was uplifted to beta)
Reporter | ||
Updated•3 years ago
|
Reporter | ||
Comment 7•3 years ago
|
||
Verified fixed with yesterday's central-as-beta simulation.
Status: RESOLVED → VERIFIED
Updated•3 years ago
|
Has Regression Range: --- → yes
You need to log in
before you can comment on or make changes to this bug.
Description
•