Wasm baseline: Avoid a temp + copy for unaligned store that is not tee_store

RESOLVED WONTFIX

Status

()

P5
normal
RESOLVED WONTFIX
2 years ago
2 years ago

People

(Reporter: lth, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

2 years ago
A corner case that needs data from *real* applications before it's worth doing.

From store() in WasmBaselineCompile.cpp:

We perform the copy on the i32 path (and allocate the temp for the copy) because we will destroy the value in the temp.  We could avoid the copy and the temp if the caller would instead preserve src when it needs to return its value as a result (for teeStore). If unaligned accesses are common it will be worthwhile to make that change, but there's no evidence yet that they will be common.
(Reporter)

Comment 1

2 years ago
Will be made obsolete by code removal in bug 1333370.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.