From 2c71f18ff7f5e1504ffeff85f643314e84e6e5d9 Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Thu, 11 Nov 2010 16:20:28 +0000 Subject: [PATCH] It's safe to sink some instructions which are not safe to speculatively execute. Make Sink's predicate more precise. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@118787 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Scalar/Sink.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/Transforms/Scalar/Sink.cpp b/lib/Transforms/Scalar/Sink.cpp index 1aca908267b..d6f1e93a812 100644 --- a/lib/Transforms/Scalar/Sink.cpp +++ b/lib/Transforms/Scalar/Sink.cpp @@ -169,7 +169,10 @@ static bool isSafeToMove(Instruction *Inst, AliasAnalysis *AA, return false; } - return Inst->isSafeToSpeculativelyExecute(); + if (isa(Inst) || isa(Inst)) + return false; + + return true; } /// SinkInstruction - Determine whether it is safe to sink the specified machine