Closed Bug 1768394 Opened 3 years ago Closed 3 months ago

OOL traps regresses some microbenchmarks (was: Regression on AWFY around 21 April mark on "float-mm.c.*" tests)

Categories

(Core :: JavaScript: WebAssembly, defect, P3)

defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: mayankleoboy1, Unassigned)

Details

Look at AWFY, on the following benchmarks :

float-mm.c-Average
float-mm.c-First
float-mm.c-Geometric
float-mm.c-Worst

Regression range : https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=201252c849f427b49422fffea6ca65e36cdade1d&tochange=c95859201259666efcd49533fe7cca1ddea5ab24

Flags: needinfo?(aionescu)

https://arewefastyet.com/win10/benchmarks/raptor-desktop-jetstream2?numDays=60

I think these are WebAssembly benchmarks. Bug 1680243 would be most likely then.

Component: JavaScript Engine: JIT → Javascript: WebAssembly
Flags: needinfo?(lhansen)

Is it me you wanted to needinfo?

Flags: needinfo?(aionescu) → needinfo?(mayankleoboy1)
Flags: needinfo?(mayankleoboy1)

Could be. gcc-loops, quicksort, and zlib all improved on the same date.

Microbenchmarks are easily perturbed, and of course this change also perturbed branch prediction because it changed code layout (that was the point), so it's not surprising if some programs are not happy about it. I can take a look later to see if there's anything actionable.

Flags: needinfo?(lhansen)
Severity: -- → N/A
Priority: -- → P3
Summary: Regression on AWFY around 21 April mark on "float-mm.c.*" tests → OOL traps regresses some microbenchmarks (was: Regression on AWFY around 21 April mark on "float-mm.c.*" tests)

The severity field is not set for this bug.
:rhunt, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(rhunt)
Severity: N/A → S4
Flags: needinfo?(rhunt)

Should we close this bug and move on?

Flags: needinfo?(rhunt)

Yeah, we should close this.

Status: NEW → RESOLVED
Closed: 3 months ago
Flags: needinfo?(rhunt)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.