forked from OSchip/llvm-project
Fix for PR340: Verifier misses malformed switch instruction
llvm-svn: 13618
This commit is contained in:
parent
6041fee872
commit
ab5aa14403
|
@ -173,6 +173,7 @@ namespace { // Anonymous namespace for class
|
||||||
void visitInstruction(Instruction &I);
|
void visitInstruction(Instruction &I);
|
||||||
void visitTerminatorInst(TerminatorInst &I);
|
void visitTerminatorInst(TerminatorInst &I);
|
||||||
void visitReturnInst(ReturnInst &RI);
|
void visitReturnInst(ReturnInst &RI);
|
||||||
|
void visitSwitchInst(SwitchInst &SI);
|
||||||
void visitSelectInst(SelectInst &SI);
|
void visitSelectInst(SelectInst &SI);
|
||||||
void visitUserOp1(Instruction &I);
|
void visitUserOp1(Instruction &I);
|
||||||
void visitUserOp2(Instruction &I) { visitUserOp1(I); }
|
void visitUserOp2(Instruction &I) { visitUserOp1(I); }
|
||||||
|
@ -362,6 +363,17 @@ void Verifier::visitReturnInst(ReturnInst &RI) {
|
||||||
visitTerminatorInst(RI);
|
visitTerminatorInst(RI);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void Verifier::visitSwitchInst(SwitchInst &SI) {
|
||||||
|
// Check to make sure that all of the constants in the switch instruction
|
||||||
|
// have the same type as the switched-on value.
|
||||||
|
const Type *SwitchTy = SI.getCondition()->getType();
|
||||||
|
for (unsigned i = 1, e = SI.getNumCases(); i != e; ++i)
|
||||||
|
Assert1(SI.getCaseValue(i)->getType() == SwitchTy,
|
||||||
|
"Switch constants must all be same type as switch value!", &SI);
|
||||||
|
|
||||||
|
visitTerminatorInst(SI);
|
||||||
|
}
|
||||||
|
|
||||||
void Verifier::visitSelectInst(SelectInst &SI) {
|
void Verifier::visitSelectInst(SelectInst &SI) {
|
||||||
Assert1(SI.getCondition()->getType() == Type::BoolTy,
|
Assert1(SI.getCondition()->getType() == Type::BoolTy,
|
||||||
"Select condition type must be bool!", &SI);
|
"Select condition type must be bool!", &SI);
|
||||||
|
|
Loading…
Reference in New Issue