wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Don't cache loads in import into the nativeFrameTracker, since when coming back from a nested tree we re-load the frame state into registers using import() based on the called tree's state, which pushes loads into the nativeFrameTracker that are relative to the inner tree's call depth. Only cache on writes now, which should be always safe.

Browse Source
This commit is contained in:
Andreas Gal 2008-08-12 23:30:49 -07:00
parent 8af146d25b
commit 91e81a361d
1 changed files with 1 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
js/src/jstracer.cpp
View File

@ -981,12 +981,10 @@ TraceRecorder::import(LIns* base, ptrdiff_t offset, jsval* p, uint8& t,
to see doubles on entry. The first op to use this slot will emit a
f2i cast which will cancel out the i2f we insert here. */
ins = lir->insLoadi(base, offset);
nativeFrameTracker.set(p, ins);
ins = lir->ins1(LIR_i2f, ins);
} else {
JS_ASSERT(isNumber(*p) == (t == JSVAL_DOUBLE));
ins = lir->insLoad(t == JSVAL_DOUBLE ? LIR_ldq : LIR_ld, base, offset);
nativeFrameTracker.set(p, ins);
}
tracker.set(p, ins);
#ifdef DEBUG
@ -1093,10 +1091,7 @@ TraceRecorder::set(jsval* p, LIns* i, bool initializing)
? -treeInfo->nativeStackBase + nativeStackOffset(p) \
: nativeGlobalOffset(p))); \
if (x->isop(LIR_ld) || x->isop(LIR_ldq)) {
ASSERT_VALID_CACHE_HIT(x->oprnd1(), x->oprnd2()->constval());
lir->insStorei(i, x->oprnd1(), x->oprnd2()->constval());
} else if (x->isop(LIR_st) || x->isop(LIR_stq)) {
if (x->isop(LIR_st) || x->isop(LIR_stq)) {
ASSERT_VALID_CACHE_HIT(x->oprnd2(), x->oprnd3()->constval());
lir->insStorei(i, x->oprnd2(), x->oprnd3()->constval());
} else {