diff options
author | Mike Pall <mike> | 2023-04-16 13:29:58 +0200 |
---|---|---|
committer | Mike Pall <mike> | 2023-04-16 13:29:58 +0200 |
commit | 8bbd58e534522a7e1ab9ee2c0d7fb723741dd089 (patch) | |
tree | fd8838460345c81252295db5b03419070c6889e1 /src | |
parent | 505e2c03de35e2718eef0d2d3660712e06dadf1f (diff) | |
parent | c7db8255e1eb59f933fac7bc9322f0e4f8ddc6e6 (diff) | |
download | luajit-8bbd58e534522a7e1ab9ee2c0d7fb723741dd089.tar.gz luajit-8bbd58e534522a7e1ab9ee2c0d7fb723741dd089.tar.bz2 luajit-8bbd58e534522a7e1ab9ee2c0d7fb723741dd089.zip |
Merge branch 'master' into v2.1
Diffstat (limited to '')
-rw-r--r-- | src/lj_opt_mem.c | 6 | ||||
-rw-r--r-- | src/lj_record.c | 2 |
2 files changed, 8 insertions, 0 deletions
diff --git a/src/lj_opt_mem.c b/src/lj_opt_mem.c index 09de2f05..455b0ca6 100644 --- a/src/lj_opt_mem.c +++ b/src/lj_opt_mem.c | |||
@@ -186,6 +186,7 @@ static TRef fwd_ahload(jit_State *J, IRRef xref) | |||
186 | fwd_aa_tab_clear(J, tab, tab)) { | 186 | fwd_aa_tab_clear(J, tab, tab)) { |
187 | /* A NEWREF with a number key may end up pointing to the array part. | 187 | /* A NEWREF with a number key may end up pointing to the array part. |
188 | ** But it's referenced from HSTORE and not found in the ASTORE chain. | 188 | ** But it's referenced from HSTORE and not found in the ASTORE chain. |
189 | ** Or a NEWREF may rehash the table and move unrelated number keys. | ||
189 | ** For now simply consider this a conflict without forwarding anything. | 190 | ** For now simply consider this a conflict without forwarding anything. |
190 | */ | 191 | */ |
191 | if (xr->o == IR_AREF) { | 192 | if (xr->o == IR_AREF) { |
@@ -196,6 +197,11 @@ static TRef fwd_ahload(jit_State *J, IRRef xref) | |||
196 | goto cselim; | 197 | goto cselim; |
197 | ref2 = newref->prev; | 198 | ref2 = newref->prev; |
198 | } | 199 | } |
200 | } else { | ||
201 | IRIns *key = IR(xr->op2); | ||
202 | if (key->o == IR_KSLOT) key = IR(key->op1); | ||
203 | if (irt_isnum(key->t) && J->chain[IR_NEWREF] > tab) | ||
204 | goto cselim; | ||
199 | } | 205 | } |
200 | /* NEWREF inhibits CSE for HREF, and dependent FLOADs from HREFK/AREF. | 206 | /* NEWREF inhibits CSE for HREF, and dependent FLOADs from HREFK/AREF. |
201 | ** But the above search for conflicting stores was limited by xref. | 207 | ** But the above search for conflicting stores was limited by xref. |
diff --git a/src/lj_record.c b/src/lj_record.c index 79ee6172..b2639cbc 100644 --- a/src/lj_record.c +++ b/src/lj_record.c | |||
@@ -1600,6 +1600,8 @@ TRef lj_record_idx(jit_State *J, RecordIndex *ix) | |||
1600 | TRef key = ix->key; | 1600 | TRef key = ix->key; |
1601 | if (tref_isinteger(key)) /* NEWREF needs a TValue as a key. */ | 1601 | if (tref_isinteger(key)) /* NEWREF needs a TValue as a key. */ |
1602 | key = emitir(IRTN(IR_CONV), key, IRCONV_NUM_INT); | 1602 | key = emitir(IRTN(IR_CONV), key, IRCONV_NUM_INT); |
1603 | else if (tref_isnumber(key) && tref_isk(key) && tvismzero(&ix->keyv)) | ||
1604 | key = lj_ir_knum_zero(J); /* Canonicalize -0.0 to +0.0. */ | ||
1603 | xref = emitir(IRT(IR_NEWREF, IRT_PGC), ix->tab, key); | 1605 | xref = emitir(IRT(IR_NEWREF, IRT_PGC), ix->tab, key); |
1604 | keybarrier = 0; /* NEWREF already takes care of the key barrier. */ | 1606 | keybarrier = 0; /* NEWREF already takes care of the key barrier. */ |
1605 | #ifdef LUAJIT_ENABLE_TABLE_BUMP | 1607 | #ifdef LUAJIT_ENABLE_TABLE_BUMP |