mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-12-17 03:30:28 +00:00
SimplifyCFG: Use ComputeMaskedBits to prune dead cases from switch instructions.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@131345 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
67f071e697
commit
10fcfb53fd
@ -20,6 +20,7 @@
|
||||
#include "llvm/DerivedTypes.h"
|
||||
#include "llvm/GlobalVariable.h"
|
||||
#include "llvm/Analysis/InstructionSimplify.h"
|
||||
#include "llvm/Analysis/ValueTracking.h"
|
||||
#include "llvm/Target/TargetData.h"
|
||||
#include "llvm/Transforms/Utils/BasicBlockUtils.h"
|
||||
#include "llvm/ADT/DenseMap.h"
|
||||
@ -2383,6 +2384,36 @@ static bool TurnSwitchRangeIntoICmp(SwitchInst *SI) {
|
||||
return true;
|
||||
}
|
||||
|
||||
/// EliminateDeadSwitchCases - Compute masked bits for the condition of a switch
|
||||
/// and use it to remove dead cases.
|
||||
static bool EliminateDeadSwitchCases(SwitchInst *SI) {
|
||||
Value *Cond = SI->getCondition();
|
||||
unsigned Bits = cast<IntegerType>(Cond->getType())->getBitWidth();
|
||||
APInt KnownZero(Bits, 0), KnownOne(Bits, 0);
|
||||
ComputeMaskedBits(Cond, APInt::getAllOnesValue(Bits), KnownZero, KnownOne);
|
||||
|
||||
// Gather dead cases.
|
||||
SmallVector<ConstantInt*, 8> DeadCases;
|
||||
for (unsigned I = 1, E = SI->getNumCases(); I != E; ++I) {
|
||||
if ((SI->getCaseValue(I)->getValue() & KnownZero) != 0 ||
|
||||
(SI->getCaseValue(I)->getValue() & KnownOne) != KnownOne) {
|
||||
DeadCases.push_back(SI->getCaseValue(I));
|
||||
DEBUG(dbgs() << "SimplifyCFG: switch case '"
|
||||
<< SI->getCaseValue(I)->getValue() << "' is dead.\n");
|
||||
}
|
||||
}
|
||||
|
||||
// Remove dead cases from the switch.
|
||||
for (unsigned I = 0, E = DeadCases.size(); I != E; ++I) {
|
||||
unsigned Case = SI->findCaseValue(DeadCases[I]);
|
||||
// Prune unused values from PHI nodes.
|
||||
SI->getSuccessor(Case)->removePredecessor(SI->getParent());
|
||||
SI->removeCase(Case);
|
||||
}
|
||||
|
||||
return !DeadCases.empty();
|
||||
}
|
||||
|
||||
bool SimplifyCFGOpt::SimplifySwitch(SwitchInst *SI) {
|
||||
// If this switch is too complex to want to look at, ignore it.
|
||||
if (!isValueEqualityComparison(SI))
|
||||
@ -2414,7 +2445,11 @@ bool SimplifyCFGOpt::SimplifySwitch(SwitchInst *SI) {
|
||||
// Try to transform the switch into an icmp and a branch.
|
||||
if (TurnSwitchRangeIntoICmp(SI))
|
||||
return SimplifyCFG(BB) | true;
|
||||
|
||||
|
||||
// Remove unreachable cases.
|
||||
if (EliminateDeadSwitchCases(SI))
|
||||
return SimplifyCFG(BB) | true;
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
|
38
test/Transforms/SimplifyCFG/switch-masked-bits.ll
Normal file
38
test/Transforms/SimplifyCFG/switch-masked-bits.ll
Normal file
@ -0,0 +1,38 @@
|
||||
; RUN: opt -S -simplifycfg < %s | FileCheck %s
|
||||
|
||||
define i32 @test1(i32 %x) nounwind {
|
||||
%i = shl i32 %x, 1
|
||||
switch i32 %i, label %a [
|
||||
i32 21, label %b
|
||||
i32 24, label %c
|
||||
]
|
||||
|
||||
a:
|
||||
ret i32 0
|
||||
b:
|
||||
ret i32 3
|
||||
c:
|
||||
ret i32 5
|
||||
; CHECK: @test1
|
||||
; CHECK: %cond = icmp eq i32 %i, 24
|
||||
; CHECK: %merge = select i1 %cond, i32 5, i32 0
|
||||
; CHECK: ret i32 %merge
|
||||
}
|
||||
|
||||
|
||||
define i32 @test2(i32 %x) nounwind {
|
||||
%i = shl i32 %x, 1
|
||||
switch i32 %i, label %a [
|
||||
i32 21, label %b
|
||||
i32 23, label %c
|
||||
]
|
||||
|
||||
a:
|
||||
ret i32 0
|
||||
b:
|
||||
ret i32 3
|
||||
c:
|
||||
ret i32 5
|
||||
; CHECK: @test2
|
||||
; CHECK: ret i32 0
|
||||
}
|
Loading…
Reference in New Issue
Block a user