1316816 - Wasm baseline: Avoid double dispatch in popI64(), popF32(), popF64()

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, defect, P5)

Description

[Lars T Hansen [:lth]]

Compare eg popI64() with popI32(): the former has a common case for ConstI64 and LocalI64 that dispatches to loadI64(), which does another tag check to discriminate between these.  popI32() has different cases for ConstI32 and LocalI32 that dispatch directly to the proper routines (loadConstI32 and loadLocalI32).

The code in popI64(), popF32(), and popF64() is a holdover from the early days; we should clean these up as popI32() was cleaned up.  This will improve compilation speed a little, though in truth the I32 case is probably the most important.

Comment 1

[(Away)]

Attachment: patch v1

Is this what you had in mind?

Comment 2

[Lars T Hansen [:lth]]

Comment on attachment 8812955 [details] [diff] [review]
patch v1

Review of attachment 8812955 [details] [diff] [review]:
-----------------------------------------------------------------

Indeed exactly like this.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6bcb02abe8fc
Refactor {load,pop}{I64,F32,F64} to avoid testing src.kind() twice. r=lth

Comment 4

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/6bcb02abe8fc