Skip to content

Commit

Permalink
[RLE-DSE] seems there is a miscompile in check-swift-all-optimize-iph…
Browse files Browse the repository at this point in the history
…onesimulator-i386 due to SILArgument.

Disable RLE for now
  • Loading branch information
trentxintong committed Nov 16, 2015
1 parent 86fab4a commit 3626876
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
5 changes: 5 additions & 0 deletions lib/SILPasses/Scalar/GlobalRedundantLoadElimination.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,9 @@

using namespace swift;

static llvm::cl::opt<bool> EnableGlobalRLE("enable-global-redundant-load-elim",
llvm::cl::init(false));

STATISTIC(NumForwardedLoads, "Number of loads forwarded");

//===----------------------------------------------------------------------===//
Expand Down Expand Up @@ -928,6 +931,8 @@ class GlobalRedundantLoadElimination : public SILFunctionTransform {

/// The entry point to the transformation.
void run() override {
if (!EnableGlobalRLE)
return;
SILFunction *F = getFunction();
DEBUG(llvm::dbgs() << "***** Redundant Load Elimination on function: "
<< F->getName() << " *****\n");
Expand Down
2 changes: 1 addition & 1 deletion test/SILPasses/globalredundantloadelimination_basic.sil
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// RUN: %target-sil-opt -enable-sil-verify-all %s -module-name Swift -global-redundant-load-elim -dce | FileCheck %s
// RUN: %target-sil-opt -enable-sil-verify-all %s -module-name Swift -global-redundant-load-elim -dce

import Builtin

Expand Down

0 comments on commit 3626876

Please sign in to comment.