detect invalid combination of sret and byval
[oota-llvm.git] / lib / VMCore / Verifier.cpp
1 //===-- Verifier.cpp - Implement the Module Verifier -------------*- C++ -*-==//
2 //
3 //                     The LLVM Compiler Infrastructure
4 //
5 // This file was developed by the LLVM research group and is distributed under
6 // the University of Illinois Open Source License. See LICENSE.TXT for details.
7 //
8 //===----------------------------------------------------------------------===//
9 //
10 // This file defines the function verifier interface, that can be used for some
11 // sanity checking of input to the system.
12 //
13 // Note that this does not provide full `Java style' security and verifications,
14 // instead it just tries to ensure that code is well-formed.
15 //
16 //  * Both of a binary operator's parameters are of the same type
17 //  * Verify that the indices of mem access instructions match other operands
18 //  * Verify that arithmetic and other things are only performed on first-class
19 //    types.  Verify that shifts & logicals only happen on integrals f.e.
20 //  * All of the constants in a switch statement are of the correct type
21 //  * The code is in valid SSA form
22 //  * It should be illegal to put a label into any other type (like a structure)
23 //    or to return one. [except constant arrays!]
24 //  * Only phi nodes can be self referential: 'add int %0, %0 ; <int>:0' is bad
25 //  * PHI nodes must have an entry for each predecessor, with no extras.
26 //  * PHI nodes must be the first thing in a basic block, all grouped together
27 //  * PHI nodes must have at least one entry
28 //  * All basic blocks should only end with terminator insts, not contain them
29 //  * The entry node to a function must not have predecessors
30 //  * All Instructions must be embedded into a basic block
31 //  * Functions cannot take a void-typed parameter
32 //  * Verify that a function's argument list agrees with it's declared type.
33 //  * It is illegal to specify a name for a void value.
34 //  * It is illegal to have a internal global value with no initializer
35 //  * It is illegal to have a ret instruction that returns a value that does not
36 //    agree with the function return value type.
37 //  * Function call argument types match the function prototype
38 //  * All other things that are tested by asserts spread about the code...
39 //
40 //===----------------------------------------------------------------------===//
41
42 #include "llvm/Analysis/Verifier.h"
43 #include "llvm/Assembly/Writer.h"
44 #include "llvm/CallingConv.h"
45 #include "llvm/Constants.h"
46 #include "llvm/Pass.h"
47 #include "llvm/Module.h"
48 #include "llvm/ModuleProvider.h"
49 #include "llvm/ParameterAttributes.h"
50 #include "llvm/DerivedTypes.h"
51 #include "llvm/InlineAsm.h"
52 #include "llvm/Instructions.h"
53 #include "llvm/Intrinsics.h"
54 #include "llvm/PassManager.h"
55 #include "llvm/Analysis/Dominators.h"
56 #include "llvm/Support/CFG.h"
57 #include "llvm/Support/InstVisitor.h"
58 #include "llvm/Support/Streams.h"
59 #include "llvm/ADT/SmallPtrSet.h"
60 #include "llvm/ADT/SmallVector.h"
61 #include "llvm/ADT/StringExtras.h"
62 #include "llvm/ADT/STLExtras.h"
63 #include "llvm/Support/Compiler.h"
64 #include <algorithm>
65 #include <sstream>
66 #include <cstdarg>
67 using namespace llvm;
68
69 namespace {  // Anonymous namespace for class
70
71   struct VISIBILITY_HIDDEN
72      Verifier : public FunctionPass, InstVisitor<Verifier> {
73     static char ID; // Pass ID, replacement for typeid
74     bool Broken;          // Is this module found to be broken?
75     bool RealPass;        // Are we not being run by a PassManager?
76     VerifierFailureAction action;
77                           // What to do if verification fails.
78     Module *Mod;          // Module we are verifying right now
79     DominatorTree *DT; // Dominator Tree, caution can be null!
80     std::stringstream msgs;  // A stringstream to collect messages
81
82     /// InstInThisBlock - when verifying a basic block, keep track of all of the
83     /// instructions we have seen so far.  This allows us to do efficient
84     /// dominance checks for the case when an instruction has an operand that is
85     /// an instruction in the same block.
86     SmallPtrSet<Instruction*, 16> InstsInThisBlock;
87
88     Verifier()
89       : FunctionPass((intptr_t)&ID), 
90       Broken(false), RealPass(true), action(AbortProcessAction),
91       DT(0), msgs( std::ios::app | std::ios::out ) {}
92     Verifier( VerifierFailureAction ctn )
93       : FunctionPass((intptr_t)&ID), 
94       Broken(false), RealPass(true), action(ctn), DT(0),
95       msgs( std::ios::app | std::ios::out ) {}
96     Verifier(bool AB )
97       : FunctionPass((intptr_t)&ID), 
98       Broken(false), RealPass(true),
99       action( AB ? AbortProcessAction : PrintMessageAction), DT(0),
100       msgs( std::ios::app | std::ios::out ) {}
101     Verifier(DominatorTree &dt)
102       : FunctionPass((intptr_t)&ID), 
103       Broken(false), RealPass(false), action(PrintMessageAction),
104       DT(&dt), msgs( std::ios::app | std::ios::out ) {}
105
106
107     bool doInitialization(Module &M) {
108       Mod = &M;
109       verifyTypeSymbolTable(M.getTypeSymbolTable());
110
111       // If this is a real pass, in a pass manager, we must abort before
112       // returning back to the pass manager, or else the pass manager may try to
113       // run other passes on the broken module.
114       if (RealPass)
115         return abortIfBroken();
116       return false;
117     }
118
119     bool runOnFunction(Function &F) {
120       // Get dominator information if we are being run by PassManager
121       if (RealPass) DT = &getAnalysis<DominatorTree>();
122
123       Mod = F.getParent();
124
125       visit(F);
126       InstsInThisBlock.clear();
127
128       // If this is a real pass, in a pass manager, we must abort before
129       // returning back to the pass manager, or else the pass manager may try to
130       // run other passes on the broken module.
131       if (RealPass)
132         return abortIfBroken();
133
134       return false;
135     }
136
137     bool doFinalization(Module &M) {
138       // Scan through, checking all of the external function's linkage now...
139       for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) {
140         visitGlobalValue(*I);
141
142         // Check to make sure function prototypes are okay.
143         if (I->isDeclaration()) visitFunction(*I);
144       }
145
146       for (Module::global_iterator I = M.global_begin(), E = M.global_end(); 
147            I != E; ++I)
148         visitGlobalVariable(*I);
149
150       for (Module::alias_iterator I = M.alias_begin(), E = M.alias_end(); 
151            I != E; ++I)
152         visitGlobalAlias(*I);
153
154       // If the module is broken, abort at this time.
155       return abortIfBroken();
156     }
157
158     virtual void getAnalysisUsage(AnalysisUsage &AU) const {
159       AU.setPreservesAll();
160       if (RealPass)
161         AU.addRequired<DominatorTree>();
162     }
163
164     /// abortIfBroken - If the module is broken and we are supposed to abort on
165     /// this condition, do so.
166     ///
167     bool abortIfBroken() {
168       if (Broken) {
169         msgs << "Broken module found, ";
170         switch (action) {
171           case AbortProcessAction:
172             msgs << "compilation aborted!\n";
173             cerr << msgs.str();
174             abort();
175           case PrintMessageAction:
176             msgs << "verification continues.\n";
177             cerr << msgs.str();
178             return false;
179           case ReturnStatusAction:
180             msgs << "compilation terminated.\n";
181             return Broken;
182         }
183       }
184       return false;
185     }
186
187
188     // Verification methods...
189     void verifyTypeSymbolTable(TypeSymbolTable &ST);
190     void visitGlobalValue(GlobalValue &GV);
191     void visitGlobalVariable(GlobalVariable &GV);
192     void visitGlobalAlias(GlobalAlias &GA);
193     void visitFunction(Function &F);
194     void visitBasicBlock(BasicBlock &BB);
195     void visitTruncInst(TruncInst &I);
196     void visitZExtInst(ZExtInst &I);
197     void visitSExtInst(SExtInst &I);
198     void visitFPTruncInst(FPTruncInst &I);
199     void visitFPExtInst(FPExtInst &I);
200     void visitFPToUIInst(FPToUIInst &I);
201     void visitFPToSIInst(FPToSIInst &I);
202     void visitUIToFPInst(UIToFPInst &I);
203     void visitSIToFPInst(SIToFPInst &I);
204     void visitIntToPtrInst(IntToPtrInst &I);
205     void visitPtrToIntInst(PtrToIntInst &I);
206     void visitBitCastInst(BitCastInst &I);
207     void visitPHINode(PHINode &PN);
208     void visitBinaryOperator(BinaryOperator &B);
209     void visitICmpInst(ICmpInst &IC);
210     void visitFCmpInst(FCmpInst &FC);
211     void visitExtractElementInst(ExtractElementInst &EI);
212     void visitInsertElementInst(InsertElementInst &EI);
213     void visitShuffleVectorInst(ShuffleVectorInst &EI);
214     void visitVAArgInst(VAArgInst &VAA) { visitInstruction(VAA); }
215     void visitCallInst(CallInst &CI);
216     void visitGetElementPtrInst(GetElementPtrInst &GEP);
217     void visitLoadInst(LoadInst &LI);
218     void visitStoreInst(StoreInst &SI);
219     void visitInstruction(Instruction &I);
220     void visitTerminatorInst(TerminatorInst &I);
221     void visitReturnInst(ReturnInst &RI);
222     void visitSwitchInst(SwitchInst &SI);
223     void visitSelectInst(SelectInst &SI);
224     void visitUserOp1(Instruction &I);
225     void visitUserOp2(Instruction &I) { visitUserOp1(I); }
226     void visitIntrinsicFunctionCall(Intrinsic::ID ID, CallInst &CI);
227
228     void VerifyIntrinsicPrototype(Intrinsic::ID ID, Function *F, ...);
229
230     void WriteValue(const Value *V) {
231       if (!V) return;
232       if (isa<Instruction>(V)) {
233         msgs << *V;
234       } else {
235         WriteAsOperand(msgs, V, true, Mod);
236         msgs << "\n";
237       }
238     }
239
240     void WriteType(const Type* T ) {
241       if ( !T ) return;
242       WriteTypeSymbolic(msgs, T, Mod );
243     }
244
245
246     // CheckFailed - A check failed, so print out the condition and the message
247     // that failed.  This provides a nice place to put a breakpoint if you want
248     // to see why something is not correct.
249     void CheckFailed(const std::string &Message,
250                      const Value *V1 = 0, const Value *V2 = 0,
251                      const Value *V3 = 0, const Value *V4 = 0) {
252       msgs << Message << "\n";
253       WriteValue(V1);
254       WriteValue(V2);
255       WriteValue(V3);
256       WriteValue(V4);
257       Broken = true;
258     }
259
260     void CheckFailed( const std::string& Message, const Value* V1,
261                       const Type* T2, const Value* V3 = 0 ) {
262       msgs << Message << "\n";
263       WriteValue(V1);
264       WriteType(T2);
265       WriteValue(V3);
266       Broken = true;
267     }
268   };
269
270   char Verifier::ID = 0;
271   RegisterPass<Verifier> X("verify", "Module Verifier");
272 } // End anonymous namespace
273
274
275 // Assert - We know that cond should be true, if not print an error message.
276 #define Assert(C, M) \
277   do { if (!(C)) { CheckFailed(M); return; } } while (0)
278 #define Assert1(C, M, V1) \
279   do { if (!(C)) { CheckFailed(M, V1); return; } } while (0)
280 #define Assert2(C, M, V1, V2) \
281   do { if (!(C)) { CheckFailed(M, V1, V2); return; } } while (0)
282 #define Assert3(C, M, V1, V2, V3) \
283   do { if (!(C)) { CheckFailed(M, V1, V2, V3); return; } } while (0)
284 #define Assert4(C, M, V1, V2, V3, V4) \
285   do { if (!(C)) { CheckFailed(M, V1, V2, V3, V4); return; } } while (0)
286
287
288 void Verifier::visitGlobalValue(GlobalValue &GV) {
289   Assert1(!GV.isDeclaration() ||
290           GV.hasExternalLinkage() ||
291           GV.hasDLLImportLinkage() ||
292           GV.hasExternalWeakLinkage() ||
293           (isa<GlobalAlias>(GV) &&
294            (GV.hasInternalLinkage() || GV.hasWeakLinkage())),
295   "Global is external, but doesn't have external or dllimport or weak linkage!",
296           &GV);
297
298   Assert1(!GV.hasDLLImportLinkage() || GV.isDeclaration(),
299           "Global is marked as dllimport, but not external", &GV);
300   
301   Assert1(!GV.hasAppendingLinkage() || isa<GlobalVariable>(GV),
302           "Only global variables can have appending linkage!", &GV);
303
304   if (GV.hasAppendingLinkage()) {
305     GlobalVariable &GVar = cast<GlobalVariable>(GV);
306     Assert1(isa<ArrayType>(GVar.getType()->getElementType()),
307             "Only global arrays can have appending linkage!", &GV);
308   }
309 }
310
311 void Verifier::visitGlobalVariable(GlobalVariable &GV) {
312   if (GV.hasInitializer())
313     Assert1(GV.getInitializer()->getType() == GV.getType()->getElementType(),
314             "Global variable initializer type does not match global "
315             "variable type!", &GV);
316
317   visitGlobalValue(GV);
318 }
319
320 void Verifier::visitGlobalAlias(GlobalAlias &GA) {
321   Assert1(!GA.getName().empty(),
322           "Alias name cannot be empty!", &GA);
323   Assert1(GA.hasExternalLinkage() || GA.hasInternalLinkage() ||
324           GA.hasWeakLinkage(),
325           "Alias should have external or external weak linkage!", &GA);
326   Assert1(GA.getType() == GA.getAliasee()->getType(),
327           "Alias and aliasee types should match!", &GA);
328   
329   if (!isa<GlobalValue>(GA.getAliasee())) {
330     const ConstantExpr *CE = dyn_cast<ConstantExpr>(GA.getAliasee());
331     Assert1(CE && CE->getOpcode() == Instruction::BitCast &&
332             isa<GlobalValue>(CE->getOperand(0)),
333             "Aliasee should be either GlobalValue or bitcast of GlobalValue",
334             &GA);
335   }
336   
337   visitGlobalValue(GA);
338 }
339
340 void Verifier::verifyTypeSymbolTable(TypeSymbolTable &ST) {
341 }
342
343 // visitFunction - Verify that a function is ok.
344 //
345 void Verifier::visitFunction(Function &F) {
346   // Check function arguments.
347   const FunctionType *FT = F.getFunctionType();
348   unsigned NumArgs = F.getArgumentList().size();
349
350   Assert2(FT->getNumParams() == NumArgs,
351           "# formal arguments must match # of arguments for function type!",
352           &F, FT);
353   Assert1(F.getReturnType()->isFirstClassType() ||
354           F.getReturnType() == Type::VoidTy,
355           "Functions cannot return aggregate values!", &F);
356
357   Assert1(!FT->isStructReturn() ||
358           (FT->getReturnType() == Type::VoidTy && 
359            FT->getNumParams() > 0 && isa<PointerType>(FT->getParamType(0))),
360           "Invalid struct-return function!", &F);
361
362   if (const ParamAttrsList *Attrs = FT->getParamAttrs()) {
363     unsigned Idx = 1;
364
365     Assert(!Attrs->paramHasAttr(0, ParamAttr::ByVal),
366            "Attribute ByVal should not apply to functions!");
367
368     for (FunctionType::param_iterator I = FT->param_begin(), 
369          E = FT->param_end(); I != E; ++I, ++Idx) {
370       if (Attrs->paramHasAttr(Idx, ParamAttr::ZExt) ||
371           Attrs->paramHasAttr(Idx, ParamAttr::SExt))
372         Assert1(FT->getParamType(Idx-1)->isInteger(),
373                 "Attribute ZExt should only apply to Integer type!", &F);
374       if (Attrs->paramHasAttr(Idx, ParamAttr::NoAlias))
375         Assert1(isa<PointerType>(FT->getParamType(Idx-1)),
376                 "Attribute NoAlias should only apply to Pointer type!", &F);
377       if (Attrs->paramHasAttr(Idx, ParamAttr::ByVal)) {
378         Assert1(isa<PointerType>(FT->getParamType(Idx-1)),
379                 "Attribute ByVal should only apply to pointer to structs!", &F);
380
381         Assert1(!Attrs->paramHasAttr(Idx, ParamAttr::StructRet),
382                 "Attributes ByVal and StructRet are incompatible!", &F);
383
384         const PointerType *Ty =
385             cast<PointerType>(FT->getParamType(Idx-1));
386         Assert1(isa<StructType>(Ty->getElementType()),
387                 "Attribute ByVal should only apply to pointer to structs!", &F);
388       }
389     }
390   }
391
392   // Check that this function meets the restrictions on this calling convention.
393   switch (F.getCallingConv()) {
394   default:
395     break;
396   case CallingConv::C:
397     break;
398   case CallingConv::Fast:
399   case CallingConv::Cold:
400   case CallingConv::X86_FastCall:
401     Assert1(!F.isVarArg(),
402             "Varargs functions must have C calling conventions!", &F);
403     break;
404   }
405   
406   // Check that the argument values match the function type for this function...
407   unsigned i = 0;
408   for (Function::arg_iterator I = F.arg_begin(), E = F.arg_end();
409        I != E; ++I, ++i) {
410     Assert2(I->getType() == FT->getParamType(i),
411             "Argument value does not match function argument type!",
412             I, FT->getParamType(i));
413     // Make sure no aggregates are passed by value.
414     Assert1(I->getType()->isFirstClassType(),
415             "Functions cannot take aggregates as arguments by value!", I);
416    }
417
418   if (!F.isDeclaration()) {
419     // Verify that this function (which has a body) is not named "llvm.*".  It
420     // is not legal to define intrinsics.
421     if (F.getName().size() >= 5)
422       Assert1(F.getName().substr(0, 5) != "llvm.",
423               "llvm intrinsics cannot be defined!", &F);
424     
425     // Check the entry node
426     BasicBlock *Entry = &F.getEntryBlock();
427     Assert1(pred_begin(Entry) == pred_end(Entry),
428             "Entry block to function must not have predecessors!", Entry);
429   }
430 }
431
432
433 // verifyBasicBlock - Verify that a basic block is well formed...
434 //
435 void Verifier::visitBasicBlock(BasicBlock &BB) {
436   InstsInThisBlock.clear();
437
438   // Ensure that basic blocks have terminators!
439   Assert1(BB.getTerminator(), "Basic Block does not have terminator!", &BB);
440
441   // Check constraints that this basic block imposes on all of the PHI nodes in
442   // it.
443   if (isa<PHINode>(BB.front())) {
444     SmallVector<BasicBlock*, 8> Preds(pred_begin(&BB), pred_end(&BB));
445     SmallVector<std::pair<BasicBlock*, Value*>, 8> Values;
446     std::sort(Preds.begin(), Preds.end());
447     PHINode *PN;
448     for (BasicBlock::iterator I = BB.begin(); (PN = dyn_cast<PHINode>(I));++I) {
449
450       // Ensure that PHI nodes have at least one entry!
451       Assert1(PN->getNumIncomingValues() != 0,
452               "PHI nodes must have at least one entry.  If the block is dead, "
453               "the PHI should be removed!", PN);
454       Assert1(PN->getNumIncomingValues() == Preds.size(),
455               "PHINode should have one entry for each predecessor of its "
456               "parent basic block!", PN);
457
458       // Get and sort all incoming values in the PHI node...
459       Values.clear();
460       Values.reserve(PN->getNumIncomingValues());
461       for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i)
462         Values.push_back(std::make_pair(PN->getIncomingBlock(i),
463                                         PN->getIncomingValue(i)));
464       std::sort(Values.begin(), Values.end());
465
466       for (unsigned i = 0, e = Values.size(); i != e; ++i) {
467         // Check to make sure that if there is more than one entry for a
468         // particular basic block in this PHI node, that the incoming values are
469         // all identical.
470         //
471         Assert4(i == 0 || Values[i].first  != Values[i-1].first ||
472                 Values[i].second == Values[i-1].second,
473                 "PHI node has multiple entries for the same basic block with "
474                 "different incoming values!", PN, Values[i].first,
475                 Values[i].second, Values[i-1].second);
476
477         // Check to make sure that the predecessors and PHI node entries are
478         // matched up.
479         Assert3(Values[i].first == Preds[i],
480                 "PHI node entries do not match predecessors!", PN,
481                 Values[i].first, Preds[i]);
482       }
483     }
484   }
485 }
486
487 void Verifier::visitTerminatorInst(TerminatorInst &I) {
488   // Ensure that terminators only exist at the end of the basic block.
489   Assert1(&I == I.getParent()->getTerminator(),
490           "Terminator found in the middle of a basic block!", I.getParent());
491   visitInstruction(I);
492 }
493
494 void Verifier::visitReturnInst(ReturnInst &RI) {
495   Function *F = RI.getParent()->getParent();
496   if (RI.getNumOperands() == 0)
497     Assert2(F->getReturnType() == Type::VoidTy,
498             "Found return instr that returns void in Function of non-void "
499             "return type!", &RI, F->getReturnType());
500   else
501     Assert2(F->getReturnType() == RI.getOperand(0)->getType(),
502             "Function return type does not match operand "
503             "type of return inst!", &RI, F->getReturnType());
504
505   // Check to make sure that the return value has necessary properties for
506   // terminators...
507   visitTerminatorInst(RI);
508 }
509
510 void Verifier::visitSwitchInst(SwitchInst &SI) {
511   // Check to make sure that all of the constants in the switch instruction
512   // have the same type as the switched-on value.
513   const Type *SwitchTy = SI.getCondition()->getType();
514   for (unsigned i = 1, e = SI.getNumCases(); i != e; ++i)
515     Assert1(SI.getCaseValue(i)->getType() == SwitchTy,
516             "Switch constants must all be same type as switch value!", &SI);
517
518   visitTerminatorInst(SI);
519 }
520
521 void Verifier::visitSelectInst(SelectInst &SI) {
522   Assert1(SI.getCondition()->getType() == Type::Int1Ty,
523           "Select condition type must be bool!", &SI);
524   Assert1(SI.getTrueValue()->getType() == SI.getFalseValue()->getType(),
525           "Select values must have identical types!", &SI);
526   Assert1(SI.getTrueValue()->getType() == SI.getType(),
527           "Select values must have same type as select instruction!", &SI);
528   visitInstruction(SI);
529 }
530
531
532 /// visitUserOp1 - User defined operators shouldn't live beyond the lifetime of
533 /// a pass, if any exist, it's an error.
534 ///
535 void Verifier::visitUserOp1(Instruction &I) {
536   Assert1(0, "User-defined operators should not live outside of a pass!", &I);
537 }
538
539 void Verifier::visitTruncInst(TruncInst &I) {
540   // Get the source and destination types
541   const Type *SrcTy = I.getOperand(0)->getType();
542   const Type *DestTy = I.getType();
543
544   // Get the size of the types in bits, we'll need this later
545   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
546   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
547
548   Assert1(SrcTy->isInteger(), "Trunc only operates on integer", &I);
549   Assert1(DestTy->isInteger(), "Trunc only produces integer", &I);
550   Assert1(SrcBitSize > DestBitSize,"DestTy too big for Trunc", &I);
551
552   visitInstruction(I);
553 }
554
555 void Verifier::visitZExtInst(ZExtInst &I) {
556   // Get the source and destination types
557   const Type *SrcTy = I.getOperand(0)->getType();
558   const Type *DestTy = I.getType();
559
560   // Get the size of the types in bits, we'll need this later
561   Assert1(SrcTy->isInteger(), "ZExt only operates on integer", &I);
562   Assert1(DestTy->isInteger(), "ZExt only produces an integer", &I);
563   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
564   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
565
566   Assert1(SrcBitSize < DestBitSize,"Type too small for ZExt", &I);
567
568   visitInstruction(I);
569 }
570
571 void Verifier::visitSExtInst(SExtInst &I) {
572   // Get the source and destination types
573   const Type *SrcTy = I.getOperand(0)->getType();
574   const Type *DestTy = I.getType();
575
576   // Get the size of the types in bits, we'll need this later
577   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
578   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
579
580   Assert1(SrcTy->isInteger(), "SExt only operates on integer", &I);
581   Assert1(DestTy->isInteger(), "SExt only produces an integer", &I);
582   Assert1(SrcBitSize < DestBitSize,"Type too small for SExt", &I);
583
584   visitInstruction(I);
585 }
586
587 void Verifier::visitFPTruncInst(FPTruncInst &I) {
588   // Get the source and destination types
589   const Type *SrcTy = I.getOperand(0)->getType();
590   const Type *DestTy = I.getType();
591   // Get the size of the types in bits, we'll need this later
592   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
593   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
594
595   Assert1(SrcTy->isFloatingPoint(),"FPTrunc only operates on FP", &I);
596   Assert1(DestTy->isFloatingPoint(),"FPTrunc only produces an FP", &I);
597   Assert1(SrcBitSize > DestBitSize,"DestTy too big for FPTrunc", &I);
598
599   visitInstruction(I);
600 }
601
602 void Verifier::visitFPExtInst(FPExtInst &I) {
603   // Get the source and destination types
604   const Type *SrcTy = I.getOperand(0)->getType();
605   const Type *DestTy = I.getType();
606
607   // Get the size of the types in bits, we'll need this later
608   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
609   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
610
611   Assert1(SrcTy->isFloatingPoint(),"FPExt only operates on FP", &I);
612   Assert1(DestTy->isFloatingPoint(),"FPExt only produces an FP", &I);
613   Assert1(SrcBitSize < DestBitSize,"DestTy too small for FPExt", &I);
614
615   visitInstruction(I);
616 }
617
618 void Verifier::visitUIToFPInst(UIToFPInst &I) {
619   // Get the source and destination types
620   const Type *SrcTy = I.getOperand(0)->getType();
621   const Type *DestTy = I.getType();
622
623   Assert1(SrcTy->isInteger(),"UInt2FP source must be integral", &I);
624   Assert1(DestTy->isFloatingPoint(),"UInt2FP result must be FP", &I);
625
626   visitInstruction(I);
627 }
628
629 void Verifier::visitSIToFPInst(SIToFPInst &I) {
630   // Get the source and destination types
631   const Type *SrcTy = I.getOperand(0)->getType();
632   const Type *DestTy = I.getType();
633
634   Assert1(SrcTy->isInteger(),"SInt2FP source must be integral", &I);
635   Assert1(DestTy->isFloatingPoint(),"SInt2FP result must be FP", &I);
636
637   visitInstruction(I);
638 }
639
640 void Verifier::visitFPToUIInst(FPToUIInst &I) {
641   // Get the source and destination types
642   const Type *SrcTy = I.getOperand(0)->getType();
643   const Type *DestTy = I.getType();
644
645   Assert1(SrcTy->isFloatingPoint(),"FP2UInt source must be FP", &I);
646   Assert1(DestTy->isInteger(),"FP2UInt result must be integral", &I);
647
648   visitInstruction(I);
649 }
650
651 void Verifier::visitFPToSIInst(FPToSIInst &I) {
652   // Get the source and destination types
653   const Type *SrcTy = I.getOperand(0)->getType();
654   const Type *DestTy = I.getType();
655
656   Assert1(SrcTy->isFloatingPoint(),"FPToSI source must be FP", &I);
657   Assert1(DestTy->isInteger(),"FP2ToI result must be integral", &I);
658
659   visitInstruction(I);
660 }
661
662 void Verifier::visitPtrToIntInst(PtrToIntInst &I) {
663   // Get the source and destination types
664   const Type *SrcTy = I.getOperand(0)->getType();
665   const Type *DestTy = I.getType();
666
667   Assert1(isa<PointerType>(SrcTy), "PtrToInt source must be pointer", &I);
668   Assert1(DestTy->isInteger(), "PtrToInt result must be integral", &I);
669
670   visitInstruction(I);
671 }
672
673 void Verifier::visitIntToPtrInst(IntToPtrInst &I) {
674   // Get the source and destination types
675   const Type *SrcTy = I.getOperand(0)->getType();
676   const Type *DestTy = I.getType();
677
678   Assert1(SrcTy->isInteger(), "IntToPtr source must be an integral", &I);
679   Assert1(isa<PointerType>(DestTy), "IntToPtr result must be a pointer",&I);
680
681   visitInstruction(I);
682 }
683
684 void Verifier::visitBitCastInst(BitCastInst &I) {
685   // Get the source and destination types
686   const Type *SrcTy = I.getOperand(0)->getType();
687   const Type *DestTy = I.getType();
688
689   // Get the size of the types in bits, we'll need this later
690   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
691   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
692
693   // BitCast implies a no-op cast of type only. No bits change.
694   // However, you can't cast pointers to anything but pointers.
695   Assert1(isa<PointerType>(DestTy) == isa<PointerType>(DestTy),
696           "Bitcast requires both operands to be pointer or neither", &I);
697   Assert1(SrcBitSize == DestBitSize, "Bitcast requies types of same width", &I);
698
699   visitInstruction(I);
700 }
701
702 /// visitPHINode - Ensure that a PHI node is well formed.
703 ///
704 void Verifier::visitPHINode(PHINode &PN) {
705   // Ensure that the PHI nodes are all grouped together at the top of the block.
706   // This can be tested by checking whether the instruction before this is
707   // either nonexistent (because this is begin()) or is a PHI node.  If not,
708   // then there is some other instruction before a PHI.
709   Assert2(&PN == &PN.getParent()->front() || 
710           isa<PHINode>(--BasicBlock::iterator(&PN)),
711           "PHI nodes not grouped at top of basic block!",
712           &PN, PN.getParent());
713
714   // Check that all of the operands of the PHI node have the same type as the
715   // result.
716   for (unsigned i = 0, e = PN.getNumIncomingValues(); i != e; ++i)
717     Assert1(PN.getType() == PN.getIncomingValue(i)->getType(),
718             "PHI node operands are not the same type as the result!", &PN);
719
720   // All other PHI node constraints are checked in the visitBasicBlock method.
721
722   visitInstruction(PN);
723 }
724
725 void Verifier::visitCallInst(CallInst &CI) {
726   Assert1(isa<PointerType>(CI.getOperand(0)->getType()),
727           "Called function must be a pointer!", &CI);
728   const PointerType *FPTy = cast<PointerType>(CI.getOperand(0)->getType());
729   Assert1(isa<FunctionType>(FPTy->getElementType()),
730           "Called function is not pointer to function type!", &CI);
731
732   const FunctionType *FTy = cast<FunctionType>(FPTy->getElementType());
733
734   // Verify that the correct number of arguments are being passed
735   if (FTy->isVarArg())
736     Assert1(CI.getNumOperands()-1 >= FTy->getNumParams(),
737             "Called function requires more parameters than were provided!",&CI);
738   else
739     Assert1(CI.getNumOperands()-1 == FTy->getNumParams(),
740             "Incorrect number of arguments passed to called function!", &CI);
741
742   // Verify that all arguments to the call match the function type...
743   for (unsigned i = 0, e = FTy->getNumParams(); i != e; ++i)
744     Assert3(CI.getOperand(i+1)->getType() == FTy->getParamType(i),
745             "Call parameter type does not match function signature!",
746             CI.getOperand(i+1), FTy->getParamType(i), &CI);
747
748   if (Function *F = CI.getCalledFunction())
749     if (Intrinsic::ID ID = (Intrinsic::ID)F->getIntrinsicID())
750       visitIntrinsicFunctionCall(ID, CI);
751
752   visitInstruction(CI);
753 }
754
755 /// visitBinaryOperator - Check that both arguments to the binary operator are
756 /// of the same type!
757 ///
758 void Verifier::visitBinaryOperator(BinaryOperator &B) {
759   Assert1(B.getOperand(0)->getType() == B.getOperand(1)->getType(),
760           "Both operands to a binary operator are not of the same type!", &B);
761
762   switch (B.getOpcode()) {
763   // Check that logical operators are only used with integral operands.
764   case Instruction::And:
765   case Instruction::Or:
766   case Instruction::Xor:
767     Assert1(B.getType()->isInteger() ||
768             (isa<VectorType>(B.getType()) && 
769              cast<VectorType>(B.getType())->getElementType()->isInteger()),
770             "Logical operators only work with integral types!", &B);
771     Assert1(B.getType() == B.getOperand(0)->getType(),
772             "Logical operators must have same type for operands and result!",
773             &B);
774     break;
775   case Instruction::Shl:
776   case Instruction::LShr:
777   case Instruction::AShr:
778     Assert1(B.getType()->isInteger(),
779             "Shift must return an integer result!", &B);
780     Assert1(B.getType() == B.getOperand(0)->getType(),
781             "Shift return type must be same as operands!", &B);
782     /* FALL THROUGH */
783   default:
784     // Arithmetic operators only work on integer or fp values
785     Assert1(B.getType() == B.getOperand(0)->getType(),
786             "Arithmetic operators must have same type for operands and result!",
787             &B);
788     Assert1(B.getType()->isInteger() || B.getType()->isFloatingPoint() ||
789             isa<VectorType>(B.getType()),
790             "Arithmetic operators must have integer, fp, or vector type!", &B);
791     break;
792   }
793
794   visitInstruction(B);
795 }
796
797 void Verifier::visitICmpInst(ICmpInst& IC) {
798   // Check that the operands are the same type
799   const Type* Op0Ty = IC.getOperand(0)->getType();
800   const Type* Op1Ty = IC.getOperand(1)->getType();
801   Assert1(Op0Ty == Op1Ty,
802           "Both operands to ICmp instruction are not of the same type!", &IC);
803   // Check that the operands are the right type
804   Assert1(Op0Ty->isInteger() || isa<PointerType>(Op0Ty),
805           "Invalid operand types for ICmp instruction", &IC);
806   visitInstruction(IC);
807 }
808
809 void Verifier::visitFCmpInst(FCmpInst& FC) {
810   // Check that the operands are the same type
811   const Type* Op0Ty = FC.getOperand(0)->getType();
812   const Type* Op1Ty = FC.getOperand(1)->getType();
813   Assert1(Op0Ty == Op1Ty,
814           "Both operands to FCmp instruction are not of the same type!", &FC);
815   // Check that the operands are the right type
816   Assert1(Op0Ty->isFloatingPoint(),
817           "Invalid operand types for FCmp instruction", &FC);
818   visitInstruction(FC);
819 }
820
821 void Verifier::visitExtractElementInst(ExtractElementInst &EI) {
822   Assert1(ExtractElementInst::isValidOperands(EI.getOperand(0),
823                                               EI.getOperand(1)),
824           "Invalid extractelement operands!", &EI);
825   visitInstruction(EI);
826 }
827
828 void Verifier::visitInsertElementInst(InsertElementInst &IE) {
829   Assert1(InsertElementInst::isValidOperands(IE.getOperand(0),
830                                              IE.getOperand(1),
831                                              IE.getOperand(2)),
832           "Invalid insertelement operands!", &IE);
833   visitInstruction(IE);
834 }
835
836 void Verifier::visitShuffleVectorInst(ShuffleVectorInst &SV) {
837   Assert1(ShuffleVectorInst::isValidOperands(SV.getOperand(0), SV.getOperand(1),
838                                              SV.getOperand(2)),
839           "Invalid shufflevector operands!", &SV);
840   Assert1(SV.getType() == SV.getOperand(0)->getType(),
841           "Result of shufflevector must match first operand type!", &SV);
842   
843   // Check to see if Mask is valid.
844   if (const ConstantVector *MV = dyn_cast<ConstantVector>(SV.getOperand(2))) {
845     for (unsigned i = 0, e = MV->getNumOperands(); i != e; ++i) {
846       Assert1(isa<ConstantInt>(MV->getOperand(i)) ||
847               isa<UndefValue>(MV->getOperand(i)),
848               "Invalid shufflevector shuffle mask!", &SV);
849     }
850   } else {
851     Assert1(isa<UndefValue>(SV.getOperand(2)) || 
852             isa<ConstantAggregateZero>(SV.getOperand(2)),
853             "Invalid shufflevector shuffle mask!", &SV);
854   }
855   
856   visitInstruction(SV);
857 }
858
859 void Verifier::visitGetElementPtrInst(GetElementPtrInst &GEP) {
860   SmallVector<Value*, 16> Idxs(GEP.idx_begin(), GEP.idx_end());
861   const Type *ElTy =
862     GetElementPtrInst::getIndexedType(GEP.getOperand(0)->getType(),
863                                       &Idxs[0], Idxs.size(), true);
864   Assert1(ElTy, "Invalid indices for GEP pointer type!", &GEP);
865   Assert2(isa<PointerType>(GEP.getType()) &&
866           cast<PointerType>(GEP.getType())->getElementType() == ElTy,
867           "GEP is not of right type for indices!", &GEP, ElTy);
868   visitInstruction(GEP);
869 }
870
871 void Verifier::visitLoadInst(LoadInst &LI) {
872   const Type *ElTy =
873     cast<PointerType>(LI.getOperand(0)->getType())->getElementType();
874   Assert2(ElTy == LI.getType(),
875           "Load result type does not match pointer operand type!", &LI, ElTy);
876   visitInstruction(LI);
877 }
878
879 void Verifier::visitStoreInst(StoreInst &SI) {
880   const Type *ElTy =
881     cast<PointerType>(SI.getOperand(1)->getType())->getElementType();
882   Assert2(ElTy == SI.getOperand(0)->getType(),
883           "Stored value type does not match pointer operand type!", &SI, ElTy);
884   visitInstruction(SI);
885 }
886
887
888 /// verifyInstruction - Verify that an instruction is well formed.
889 ///
890 void Verifier::visitInstruction(Instruction &I) {
891   BasicBlock *BB = I.getParent();
892   Assert1(BB, "Instruction not embedded in basic block!", &I);
893
894   if (!isa<PHINode>(I)) {   // Check that non-phi nodes are not self referential
895     for (Value::use_iterator UI = I.use_begin(), UE = I.use_end();
896          UI != UE; ++UI)
897       Assert1(*UI != (User*)&I ||
898               !DT->dominates(&BB->getParent()->getEntryBlock(), BB),
899               "Only PHI nodes may reference their own value!", &I);
900   }
901
902   // Check that void typed values don't have names
903   Assert1(I.getType() != Type::VoidTy || !I.hasName(),
904           "Instruction has a name, but provides a void value!", &I);
905
906   // Check that the return value of the instruction is either void or a legal
907   // value type.
908   Assert1(I.getType() == Type::VoidTy || I.getType()->isFirstClassType(),
909           "Instruction returns a non-scalar type!", &I);
910
911   // Check that all uses of the instruction, if they are instructions
912   // themselves, actually have parent basic blocks.  If the use is not an
913   // instruction, it is an error!
914   for (User::use_iterator UI = I.use_begin(), UE = I.use_end();
915        UI != UE; ++UI) {
916     Assert1(isa<Instruction>(*UI), "Use of instruction is not an instruction!",
917             *UI);
918     Instruction *Used = cast<Instruction>(*UI);
919     Assert2(Used->getParent() != 0, "Instruction referencing instruction not"
920             " embeded in a basic block!", &I, Used);
921   }
922
923   for (unsigned i = 0, e = I.getNumOperands(); i != e; ++i) {
924     Assert1(I.getOperand(i) != 0, "Instruction has null operand!", &I);
925
926     // Check to make sure that only first-class-values are operands to
927     // instructions.
928     Assert1(I.getOperand(i)->getType()->isFirstClassType(),
929             "Instruction operands must be first-class values!", &I);
930   
931     if (Function *F = dyn_cast<Function>(I.getOperand(i))) {
932       // Check to make sure that the "address of" an intrinsic function is never
933       // taken.
934       Assert1(!F->isIntrinsic() || (i == 0 && isa<CallInst>(I)),
935               "Cannot take the address of an intrinsic!", &I);
936       Assert1(F->getParent() == Mod, "Referencing function in another module!",
937               &I);
938     } else if (BasicBlock *OpBB = dyn_cast<BasicBlock>(I.getOperand(i))) {
939       Assert1(OpBB->getParent() == BB->getParent(),
940               "Referring to a basic block in another function!", &I);
941     } else if (Argument *OpArg = dyn_cast<Argument>(I.getOperand(i))) {
942       Assert1(OpArg->getParent() == BB->getParent(),
943               "Referring to an argument in another function!", &I);
944     } else if (GlobalValue *GV = dyn_cast<GlobalValue>(I.getOperand(i))) {
945       Assert1(GV->getParent() == Mod, "Referencing global in another module!",
946               &I);
947     } else if (Instruction *Op = dyn_cast<Instruction>(I.getOperand(i))) {
948       BasicBlock *OpBlock = Op->getParent();
949
950       // Check that a definition dominates all of its uses.
951       if (!isa<PHINode>(I)) {
952         // Invoke results are only usable in the normal destination, not in the
953         // exceptional destination.
954         if (InvokeInst *II = dyn_cast<InvokeInst>(Op)) {
955           OpBlock = II->getNormalDest();
956           
957           Assert2(OpBlock != II->getUnwindDest(),
958                   "No uses of invoke possible due to dominance structure!",
959                   Op, II);
960           
961           // If the normal successor of an invoke instruction has multiple
962           // predecessors, then the normal edge from the invoke is critical, so
963           // the invoke value can only be live if the destination block
964           // dominates all of it's predecessors (other than the invoke) or if
965           // the invoke value is only used by a phi in the successor.
966           if (!OpBlock->getSinglePredecessor() &&
967               DT->dominates(&BB->getParent()->getEntryBlock(), BB)) {
968             // The first case we allow is if the use is a PHI operand in the
969             // normal block, and if that PHI operand corresponds to the invoke's
970             // block.
971             bool Bad = true;
972             if (PHINode *PN = dyn_cast<PHINode>(&I))
973               if (PN->getParent() == OpBlock &&
974                   PN->getIncomingBlock(i/2) == Op->getParent())
975                 Bad = false;
976             
977             // If it is used by something non-phi, then the other case is that
978             // 'OpBlock' dominates all of its predecessors other than the
979             // invoke.  In this case, the invoke value can still be used.
980             if (Bad) {
981               Bad = false;
982               for (pred_iterator PI = pred_begin(OpBlock),
983                    E = pred_end(OpBlock); PI != E; ++PI) {
984                 if (*PI != II->getParent() && !DT->dominates(OpBlock, *PI)) {
985                   Bad = true;
986                   break;
987                 }
988               }
989             }
990             Assert2(!Bad,
991                     "Invoke value defined on critical edge but not dead!", &I,
992                     Op);
993           }
994         } else if (OpBlock == BB) {
995           // If they are in the same basic block, make sure that the definition
996           // comes before the use.
997           Assert2(InstsInThisBlock.count(Op) ||
998                   !DT->dominates(&BB->getParent()->getEntryBlock(), BB),
999                   "Instruction does not dominate all uses!", Op, &I);
1000         }
1001
1002         // Definition must dominate use unless use is unreachable!
1003         Assert2(DT->dominates(OpBlock, BB) ||
1004                 !DT->dominates(&BB->getParent()->getEntryBlock(), BB),
1005                 "Instruction does not dominate all uses!", Op, &I);
1006       } else {
1007         // PHI nodes are more difficult than other nodes because they actually
1008         // "use" the value in the predecessor basic blocks they correspond to.
1009         BasicBlock *PredBB = cast<BasicBlock>(I.getOperand(i+1));
1010         Assert2(DT->dominates(OpBlock, PredBB) ||
1011                 !DT->dominates(&BB->getParent()->getEntryBlock(), PredBB),
1012                 "Instruction does not dominate all uses!", Op, &I);
1013       }
1014     } else if (isa<InlineAsm>(I.getOperand(i))) {
1015       Assert1(i == 0 && isa<CallInst>(I),
1016               "Cannot take the address of an inline asm!", &I);
1017     }
1018   }
1019   InstsInThisBlock.insert(&I);
1020 }
1021
1022 /// visitIntrinsicFunction - Allow intrinsics to be verified in different ways.
1023 ///
1024 void Verifier::visitIntrinsicFunctionCall(Intrinsic::ID ID, CallInst &CI) {
1025   Function *IF = CI.getCalledFunction();
1026   Assert1(IF->isDeclaration(), "Intrinsic functions should never be defined!",
1027           IF);
1028   
1029 #define GET_INTRINSIC_VERIFIER
1030 #include "llvm/Intrinsics.gen"
1031 #undef GET_INTRINSIC_VERIFIER
1032 }
1033
1034 /// VerifyIntrinsicPrototype - TableGen emits calls to this function into
1035 /// Intrinsics.gen.  This implements a little state machine that verifies the
1036 /// prototype of intrinsics.
1037 void Verifier::VerifyIntrinsicPrototype(Intrinsic::ID ID, Function *F, ...) {
1038   va_list VA;
1039   va_start(VA, F);
1040   
1041   const FunctionType *FTy = F->getFunctionType();
1042   
1043   // For overloaded intrinsics, the Suffix of the function name must match the
1044   // types of the arguments. This variable keeps track of the expected
1045   // suffix, to be checked at the end.
1046   std::string Suffix;
1047
1048   // Note that "arg#0" is the return type.
1049   for (unsigned ArgNo = 0; 1; ++ArgNo) {
1050     int TypeID = va_arg(VA, int);
1051
1052     if (TypeID == -2) {
1053       break;
1054     }
1055
1056     if (TypeID == -1) {
1057       if (ArgNo != FTy->getNumParams()+1)
1058         CheckFailed("Intrinsic prototype has too many arguments!", F);
1059       break;
1060     }
1061
1062     if (ArgNo == FTy->getNumParams()+1) {
1063       CheckFailed("Intrinsic prototype has too few arguments!", F);
1064       break;
1065     }
1066     
1067     const Type *Ty;
1068     if (ArgNo == 0)
1069       Ty = FTy->getReturnType();
1070     else
1071       Ty = FTy->getParamType(ArgNo-1);
1072     
1073     if (TypeID != Ty->getTypeID()) {
1074       if (ArgNo == 0)
1075         CheckFailed("Intrinsic prototype has incorrect result type!", F);
1076       else
1077         CheckFailed("Intrinsic parameter #" + utostr(ArgNo-1) + " is wrong!",F);
1078       break;
1079     }
1080
1081     if (TypeID == Type::IntegerTyID) {
1082       unsigned ExpectedBits = (unsigned) va_arg(VA, int);
1083       unsigned GotBits = cast<IntegerType>(Ty)->getBitWidth();
1084       if (ExpectedBits == 0) {
1085         Suffix += ".i" + utostr(GotBits);
1086       } else if (GotBits != ExpectedBits) {
1087         std::string bitmsg = " Expected " + utostr(ExpectedBits) + " but got "+
1088                              utostr(GotBits) + " bits.";
1089         if (ArgNo == 0)
1090           CheckFailed("Intrinsic prototype has incorrect integer result width!"
1091                       + bitmsg, F);
1092         else
1093           CheckFailed("Intrinsic parameter #" + utostr(ArgNo-1) + " has "
1094                       "incorrect integer width!" + bitmsg, F);
1095         break;
1096       }
1097       // Check some constraints on various intrinsics.
1098       switch (ID) {
1099         default: break; // Not everything needs to be checked.
1100         case Intrinsic::bswap:
1101           if (GotBits < 16 || GotBits % 16 != 0)
1102             CheckFailed("Intrinsic requires even byte width argument", F);
1103           /* FALL THROUGH */
1104         case Intrinsic::part_set:
1105         case Intrinsic::part_select:
1106           if (ArgNo == 1) {
1107             unsigned ResultBits = 
1108               cast<IntegerType>(FTy->getReturnType())->getBitWidth();
1109             if (GotBits != ResultBits)
1110               CheckFailed("Intrinsic requires the bit widths of the first "
1111                           "parameter and the result to match", F);
1112           }
1113           break;
1114       }
1115     } else if (TypeID == Type::VectorTyID) {
1116       // If this is a vector argument, verify the number and type of elements.
1117       const VectorType *PTy = cast<VectorType>(Ty);
1118       int ElemTy = va_arg(VA, int);
1119       if (ElemTy != PTy->getElementType()->getTypeID()) {
1120         CheckFailed("Intrinsic prototype has incorrect vector element type!",
1121                     F);
1122         break;
1123       }
1124       if (ElemTy == Type::IntegerTyID) {
1125         unsigned NumBits = (unsigned)va_arg(VA, int);
1126         unsigned ExpectedBits = 
1127           cast<IntegerType>(PTy->getElementType())->getBitWidth();
1128         if (NumBits != ExpectedBits) {
1129           CheckFailed("Intrinsic prototype has incorrect vector element type!",
1130                       F);
1131           break;
1132         }
1133       }
1134       if ((unsigned)va_arg(VA, int) != PTy->getNumElements()) {
1135         CheckFailed("Intrinsic prototype has incorrect number of "
1136                     "vector elements!",F);
1137           break;
1138       }
1139     }
1140   }
1141
1142   va_end(VA);
1143
1144   // If we computed a Suffix then the intrinsic is overloaded and we need to 
1145   // make sure that the name of the function is correct. We add the suffix to
1146   // the name of the intrinsic and compare against the given function name. If
1147   // they are not the same, the function name is invalid. This ensures that
1148   // overloading of intrinsics uses a sane and consistent naming convention.
1149   if (!Suffix.empty()) {
1150     std::string Name(Intrinsic::getName(ID));
1151     if (Name + Suffix != F->getName())
1152       CheckFailed("Overloaded intrinsic has incorrect suffix: '" +
1153                   F->getName().substr(Name.length()) + "'. It should be '" +
1154                   Suffix + "'", F);
1155   }
1156 }
1157
1158
1159 //===----------------------------------------------------------------------===//
1160 //  Implement the public interfaces to this file...
1161 //===----------------------------------------------------------------------===//
1162
1163 FunctionPass *llvm::createVerifierPass(VerifierFailureAction action) {
1164   return new Verifier(action);
1165 }
1166
1167
1168 // verifyFunction - Create
1169 bool llvm::verifyFunction(const Function &f, VerifierFailureAction action) {
1170   Function &F = const_cast<Function&>(f);
1171   assert(!F.isDeclaration() && "Cannot verify external functions");
1172
1173   FunctionPassManager FPM(new ExistingModuleProvider(F.getParent()));
1174   Verifier *V = new Verifier(action);
1175   FPM.add(V);
1176   FPM.run(F);
1177   return V->Broken;
1178 }
1179
1180 /// verifyModule - Check a module for errors, printing messages on stderr.
1181 /// Return true if the module is corrupt.
1182 ///
1183 bool llvm::verifyModule(const Module &M, VerifierFailureAction action,
1184                         std::string *ErrorInfo) {
1185   PassManager PM;
1186   Verifier *V = new Verifier(action);
1187   PM.add(V);
1188   PM.run((Module&)M);
1189   
1190   if (ErrorInfo && V->Broken)
1191     *ErrorInfo = V->msgs.str();
1192   return V->Broken;
1193 }
1194
1195 // vim: sw=2