Skip to content

Commit e2187de

Browse files
Alisha Nayakmeta-codesync[bot]
authored andcommitted
Back out "Sink non-escaping PrimitiveBox onto deopt paths"
Summary: Original commit changeset: 502d54a7199a Original Phabricator Diff: D110384610 Mitigation for S683263 Reviewed By: alexmalyshev Differential Revision: D110895060 fbshipit-source-id: a0e1934fa6ad3add1f6e5b4b5011801673dc076a
1 parent b324a3e commit e2187de

6 files changed

Lines changed: 0 additions & 127 deletions

File tree

cinderx/Jit/compiler.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
#include "cinderx/Jit/hir/printer.h"
2121
#include "cinderx/Jit/hir/refcount_insertion.h"
2222
#include "cinderx/Jit/hir/simplify.h"
23-
#include "cinderx/Jit/hir/sink_primitive_box.h"
2423
#include "cinderx/Jit/hir/ssa.h"
2524
#include "cinderx/Jit/hir/stats.h"
2625
#include "cinderx/Jit/jit_time_log.h"
@@ -108,7 +107,6 @@ void Compiler::runPasses(
108107
hir::BuiltinLoadMethodElimination{}, PassConfig::kBuiltinLoadMethodElim);
109108
runPassIf(hir::Simplify{}, PassConfig::kSimplify);
110109
runPassIf(hir::CleanCFG{}, PassConfig::kCleanCFG);
111-
runPassIf(hir::SinkPrimitiveBox{}, PassConfig::kSinkPrimitiveBox);
112110
runPassIf(hir::DeadCodeElimination{}, PassConfig::kDeadCodeElim);
113111
runPassIf(hir::CleanCFG{}, PassConfig::kCleanCFG);
114112

@@ -166,7 +164,6 @@ PassConfig createConfig() {
166164
set(hir_opts.insert_update_prev_instr, PassConfig::kInsertUpdatePrevInstr);
167165
set(hir_opts.phi_elim, PassConfig::kPhiElim);
168166
set(hir_opts.simplify, PassConfig::kSimplify);
169-
set(hir_opts.sink_primitive_box, PassConfig::kSinkPrimitiveBox);
170167

171168
return static_cast<PassConfig>(result);
172169
}

cinderx/Jit/compiler.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ enum PassConfig : uint64_t {
3838
kPhiElim = 1 << 7,
3939
kSimplify = 1 << 8,
4040
kInsertUpdatePrevInstr = 1 << 9,
41-
kSinkPrimitiveBox = 1 << 10,
4241

4342
// Run all the passes.
4443
kAll = ~uint64_t{0},

cinderx/Jit/config.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,6 @@ struct HIROptimizations {
4343
bool insert_update_prev_instr{true};
4444
bool phi_elim{true};
4545
bool simplify{true};
46-
bool sink_primitive_box{true};
4746
};
4847

4948
// List of LIR optimization passes to run.

cinderx/Jit/hir/sink_primitive_box.cpp

Lines changed: 0 additions & 88 deletions
This file was deleted.

cinderx/Jit/hir/sink_primitive_box.h

Lines changed: 0 additions & 29 deletions
This file was deleted.

cinderx/Jit/pyjit.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -564,11 +564,6 @@ FlagProcessor initFlagProcessor() {
564564
"CINDERX_JIT_PHI_ELIM");
565565
HIR_OPTIMIZATION_OPTION(
566566
"simplify", simplify, "cinderx-jit-simplify", "CINDERX_JIT_SIMPLIFY");
567-
HIR_OPTIMIZATION_OPTION(
568-
"sink primitive box",
569-
sink_primitive_box,
570-
"cinderx-jit-sink-primitive-box",
571-
"CINDERX_JIT_SINK_PRIMITIVE_BOX");
572567

573568
flag_processor.addOption(
574569
"cinderx-jit-simplify-iteration-limit",

0 commit comments

Comments
 (0)