diff options
| author | Louis Gerbarg <lgg@apple.com> | 2014-05-09 17:02:46 +0000 |
|---|---|---|
| committer | Louis Gerbarg <lgg@apple.com> | 2014-05-09 17:02:46 +0000 |
| commit | c0adfbb49d674dc04f39536151f05cc6a5e9e5fb (patch) | |
| tree | f6a64323ac2df81442e21472d0cfec5a325a4917 /lib | |
| parent | 3f26d366a48c0c1efd61837b26e09c598cd574dd (diff) | |
| download | llvm-c0adfbb49d674dc04f39536151f05cc6a5e9e5fb.tar.gz llvm-c0adfbb49d674dc04f39536151f05cc6a5e9e5fb.tar.bz2 llvm-c0adfbb49d674dc04f39536151f05cc6a5e9e5fb.tar.xz | |
Add ExtractValue instruction to SimplifyCFG's ComputeSpeculationCost
Since ExtractValue is not included in ComputeSpeculationCost CFGs containing
ExtractValueInsts cannot be simplified. In particular this interacts with
InstCombineCompare's tendency to insert add.with.overflow intrinsics for
certain idiomatic math operations, preventing optimization.
This patch adds ExtractValue to the ComputeSpeculationCost. Test case included
rdar://14853450
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@208434 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/Transforms/Utils/SimplifyCFG.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/Transforms/Utils/SimplifyCFG.cpp b/lib/Transforms/Utils/SimplifyCFG.cpp index cb747f6f51..bfc7f4ace7 100644 --- a/lib/Transforms/Utils/SimplifyCFG.cpp +++ b/lib/Transforms/Utils/SimplifyCFG.cpp @@ -213,6 +213,7 @@ static unsigned ComputeSpeculationCost(const User *I) { if (!cast<GEPOperator>(I)->hasAllConstantIndices()) return UINT_MAX; return 1; + case Instruction::ExtractValue: case Instruction::Load: case Instruction::Add: case Instruction::Sub: |