From 10b359ccb60de3114f262d2a204f54c532758cfc Mon Sep 17 00:00:00 2001 From: Devang Patel Date: Thu, 4 Sep 2008 18:55:13 +0000 Subject: [PATCH] Do not unswitch if the function notes say we're optimizing this function for size. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@55786 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Scalar/LoopUnswitch.cpp | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/lib/Transforms/Scalar/LoopUnswitch.cpp b/lib/Transforms/Scalar/LoopUnswitch.cpp index 372e6e4f834..f9046a1a055 100644 --- a/lib/Transforms/Scalar/LoopUnswitch.cpp +++ b/lib/Transforms/Scalar/LoopUnswitch.cpp @@ -417,6 +417,13 @@ unsigned LoopUnswitch::getLoopUnswitchCost(Value *LIC) { /// LoopCond == Val to simplify the loop. If we decide that this is profitable, /// unswitch the loop, reprocess the pieces, then return true. bool LoopUnswitch::UnswitchIfProfitable(Value *LoopCond, Constant *Val){ + + Function *F = loopHeader->getParent(); + + // Do not unswitch if the function is optimized for size. + if (F->getNotes() & FN_NOTE_OptimizeForSize) + return false; + // Check to see if it would be profitable to unswitch current loop. unsigned Cost = getLoopUnswitchCost(LoopCond); @@ -445,7 +452,6 @@ bool LoopUnswitch::UnswitchIfProfitable(Value *LoopCond, Constant *Val){ } // FIXME: Reconstruct dom info, because it is not preserved properly. - Function *F = loopHeader->getParent(); if (DT) DT->runOnFunction(*F); if (DF)