summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMike Pall <mike>2023-04-16 13:26:18 +0200
committerMike Pall <mike>2023-04-16 13:26:18 +0200
commitc7db8255e1eb59f933fac7bc9322f0e4f8ddc6e6 (patch)
tree863ed6206f8d80996209c8017386336523c57334
parent96fc114a7a3be3fd2c227d5a0ac53aa50cfb85d1 (diff)
downloadluajit2-v2.0.tar.gz
Fix TDUP load forwarding after table rehash.HEADv2.0master
Reported by Sergey Kaplun. #980
-rw-r--r--src/lj_opt_mem.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/lj_opt_mem.c b/src/lj_opt_mem.c
index feec6bb7..5b1ad898 100644
--- a/src/lj_opt_mem.c
+++ b/src/lj_opt_mem.c
@@ -154,6 +154,7 @@ static TRef fwd_ahload(jit_State *J, IRRef xref)
if (ir->o == IR_TNEW || (ir->o == IR_TDUP && irref_isk(xr->op2))) {
/* A NEWREF with a number key may end up pointing to the array part.
** But it's referenced from HSTORE and not found in the ASTORE chain.
+ ** Or a NEWREF may rehash the table and move unrelated number keys.
** For now simply consider this a conflict without forwarding anything.
*/
if (xr->o == IR_AREF) {
@@ -164,6 +165,11 @@ static TRef fwd_ahload(jit_State *J, IRRef xref)
goto cselim;
ref2 = newref->prev;
}
+ } else {
+ IRIns *key = IR(xr->op2);
+ if (key->o == IR_KSLOT) key = IR(key->op1);
+ if (irt_isnum(key->t) && J->chain[IR_NEWREF] > tab)
+ goto cselim;
}
/* NEWREF inhibits CSE for HREF, and dependent FLOADs from HREFK/AREF.
** But the above search for conflicting stores was limited by xref.