Skip to content

Commit 4f30736

Browse files
committed
Try to reduce the cost for compressed displacement support
1 parent 5344cbd commit 4f30736

File tree

7 files changed

+217
-173
lines changed

7 files changed

+217
-173
lines changed

src/coreclr/jit/codegenxarch.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9531,7 +9531,7 @@ void CodeGen::genAmd64EmitterUnitTestsAvx10v2()
95319531

95329532
theEmitter->emitIns_R_R(INS_vcvttps2ibs, EA_16BYTE, REG_XMM0, REG_XMM1);
95339533
theEmitter->emitIns_R_R(INS_vcvttps2ibs, EA_32BYTE, REG_XMM0, REG_XMM1);
9534-
theEmitter->emitIns_R_R(INS_vcvttps2ibs, EA_32BYTE, REG_XMM0, REG_XMM1, INS_OPTS_EVEX_eb_er_rd);
9534+
theEmitter->emitIns_R_R(INS_vcvttps2ibs, EA_32BYTE, REG_XMM0, REG_XMM1, INS_OPTS_EVEX_er_rd);
95359535
theEmitter->emitIns_R_R(INS_vcvttps2ibs, EA_64BYTE, REG_XMM0, REG_XMM1);
95369536

95379537
theEmitter->emitIns_R_R(INS_vcvttps2iubs, EA_16BYTE, REG_XMM0, REG_XMM1);

src/coreclr/jit/emit.h

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -844,9 +844,7 @@ class emitter
844844
unsigned _idCustom5 : 1;
845845
unsigned _idCustom6 : 1;
846846

847-
#define _idEvexbContext \
848-
(_idCustom6 << 1) | _idCustom5 /* Evex.b: embedded broadcast, embedded rounding, embedded SAE \
849-
*/
847+
#define _idEvexbContext (_idCustom6 << 1) | _idCustom5 /* Evex.b: embedded broadcast, rounding, SAE */
850848
#define _idEvexNdContext _idCustom5 /* bits used for the APX-EVEX.nd context for promoted legacy instructions */
851849
#define _idEvexNfContext _idCustom6 /* bits used for the APX-EVEX.nf context for promoted legacy/vex instructions */
852850

@@ -1737,7 +1735,6 @@ class emitter
17371735
void idSetEvexbContext(insOpts instOptions)
17381736
{
17391737
assert(!idIsEvexbContextSet());
1740-
assert(idGetEvexbContext() == 0);
17411738
unsigned value = static_cast<unsigned>(instOptions & INS_OPTS_EVEX_b_MASK);
17421739

17431740
_idCustom5 = ((value >> 0) & 1);

0 commit comments

Comments
 (0)