Use DominatorTree instead of ETForest.
[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     for (FunctionType::param_iterator I = FT->param_begin(), 
365          E = FT->param_end(); I != E; ++I, ++Idx) {
366       if (Attrs->paramHasAttr(Idx, ParamAttr::ZExt) ||
367           Attrs->paramHasAttr(Idx, ParamAttr::SExt))
368         Assert1(FT->getParamType(Idx-1)->isInteger(),
369                 "Attribute ZExt should only apply to Integer type!", &F);
370       if (Attrs->paramHasAttr(Idx, ParamAttr::NoAlias))
371         Assert1(isa<PointerType>(FT->getParamType(Idx-1)),
372                 "Attribute NoAlias should only apply to Pointer type!", &F);
373     }
374   }
375
376   // Check that this function meets the restrictions on this calling convention.
377   switch (F.getCallingConv()) {
378   default:
379     break;
380   case CallingConv::C:
381     break;
382   case CallingConv::Fast:
383   case CallingConv::Cold:
384   case CallingConv::X86_FastCall:
385     Assert1(!F.isVarArg(),
386             "Varargs functions must have C calling conventions!", &F);
387     break;
388   }
389   
390   // Check that the argument values match the function type for this function...
391   unsigned i = 0;
392   for (Function::arg_iterator I = F.arg_begin(), E = F.arg_end();
393        I != E; ++I, ++i) {
394     Assert2(I->getType() == FT->getParamType(i),
395             "Argument value does not match function argument type!",
396             I, FT->getParamType(i));
397     // Make sure no aggregates are passed by value.
398     Assert1(I->getType()->isFirstClassType(),
399             "Functions cannot take aggregates as arguments by value!", I);
400    }
401
402   if (!F.isDeclaration()) {
403     // Verify that this function (which has a body) is not named "llvm.*".  It
404     // is not legal to define intrinsics.
405     if (F.getName().size() >= 5)
406       Assert1(F.getName().substr(0, 5) != "llvm.",
407               "llvm intrinsics cannot be defined!", &F);
408     
409     // Check the entry node
410     BasicBlock *Entry = &F.getEntryBlock();
411     Assert1(pred_begin(Entry) == pred_end(Entry),
412             "Entry block to function must not have predecessors!", Entry);
413   }
414 }
415
416
417 // verifyBasicBlock - Verify that a basic block is well formed...
418 //
419 void Verifier::visitBasicBlock(BasicBlock &BB) {
420   InstsInThisBlock.clear();
421
422   // Ensure that basic blocks have terminators!
423   Assert1(BB.getTerminator(), "Basic Block does not have terminator!", &BB);
424
425   // Check constraints that this basic block imposes on all of the PHI nodes in
426   // it.
427   if (isa<PHINode>(BB.front())) {
428     SmallVector<BasicBlock*, 8> Preds(pred_begin(&BB), pred_end(&BB));
429     SmallVector<std::pair<BasicBlock*, Value*>, 8> Values;
430     std::sort(Preds.begin(), Preds.end());
431     PHINode *PN;
432     for (BasicBlock::iterator I = BB.begin(); (PN = dyn_cast<PHINode>(I));++I) {
433
434       // Ensure that PHI nodes have at least one entry!
435       Assert1(PN->getNumIncomingValues() != 0,
436               "PHI nodes must have at least one entry.  If the block is dead, "
437               "the PHI should be removed!", PN);
438       Assert1(PN->getNumIncomingValues() == Preds.size(),
439               "PHINode should have one entry for each predecessor of its "
440               "parent basic block!", PN);
441
442       // Get and sort all incoming values in the PHI node...
443       Values.clear();
444       Values.reserve(PN->getNumIncomingValues());
445       for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i)
446         Values.push_back(std::make_pair(PN->getIncomingBlock(i),
447                                         PN->getIncomingValue(i)));
448       std::sort(Values.begin(), Values.end());
449
450       for (unsigned i = 0, e = Values.size(); i != e; ++i) {
451         // Check to make sure that if there is more than one entry for a
452         // particular basic block in this PHI node, that the incoming values are
453         // all identical.
454         //
455         Assert4(i == 0 || Values[i].first  != Values[i-1].first ||
456                 Values[i].second == Values[i-1].second,
457                 "PHI node has multiple entries for the same basic block with "
458                 "different incoming values!", PN, Values[i].first,
459                 Values[i].second, Values[i-1].second);
460
461         // Check to make sure that the predecessors and PHI node entries are
462         // matched up.
463         Assert3(Values[i].first == Preds[i],
464                 "PHI node entries do not match predecessors!", PN,
465                 Values[i].first, Preds[i]);
466       }
467     }
468   }
469 }
470
471 void Verifier::visitTerminatorInst(TerminatorInst &I) {
472   // Ensure that terminators only exist at the end of the basic block.
473   Assert1(&I == I.getParent()->getTerminator(),
474           "Terminator found in the middle of a basic block!", I.getParent());
475   visitInstruction(I);
476 }
477
478 void Verifier::visitReturnInst(ReturnInst &RI) {
479   Function *F = RI.getParent()->getParent();
480   if (RI.getNumOperands() == 0)
481     Assert2(F->getReturnType() == Type::VoidTy,
482             "Found return instr that returns void in Function of non-void "
483             "return type!", &RI, F->getReturnType());
484   else
485     Assert2(F->getReturnType() == RI.getOperand(0)->getType(),
486             "Function return type does not match operand "
487             "type of return inst!", &RI, F->getReturnType());
488
489   // Check to make sure that the return value has necessary properties for
490   // terminators...
491   visitTerminatorInst(RI);
492 }
493
494 void Verifier::visitSwitchInst(SwitchInst &SI) {
495   // Check to make sure that all of the constants in the switch instruction
496   // have the same type as the switched-on value.
497   const Type *SwitchTy = SI.getCondition()->getType();
498   for (unsigned i = 1, e = SI.getNumCases(); i != e; ++i)
499     Assert1(SI.getCaseValue(i)->getType() == SwitchTy,
500             "Switch constants must all be same type as switch value!", &SI);
501
502   visitTerminatorInst(SI);
503 }
504
505 void Verifier::visitSelectInst(SelectInst &SI) {
506   Assert1(SI.getCondition()->getType() == Type::Int1Ty,
507           "Select condition type must be bool!", &SI);
508   Assert1(SI.getTrueValue()->getType() == SI.getFalseValue()->getType(),
509           "Select values must have identical types!", &SI);
510   Assert1(SI.getTrueValue()->getType() == SI.getType(),
511           "Select values must have same type as select instruction!", &SI);
512   visitInstruction(SI);
513 }
514
515
516 /// visitUserOp1 - User defined operators shouldn't live beyond the lifetime of
517 /// a pass, if any exist, it's an error.
518 ///
519 void Verifier::visitUserOp1(Instruction &I) {
520   Assert1(0, "User-defined operators should not live outside of a pass!", &I);
521 }
522
523 void Verifier::visitTruncInst(TruncInst &I) {
524   // Get the source and destination types
525   const Type *SrcTy = I.getOperand(0)->getType();
526   const Type *DestTy = I.getType();
527
528   // Get the size of the types in bits, we'll need this later
529   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
530   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
531
532   Assert1(SrcTy->isInteger(), "Trunc only operates on integer", &I);
533   Assert1(DestTy->isInteger(), "Trunc only produces integer", &I);
534   Assert1(SrcBitSize > DestBitSize,"DestTy too big for Trunc", &I);
535
536   visitInstruction(I);
537 }
538
539 void Verifier::visitZExtInst(ZExtInst &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   Assert1(SrcTy->isInteger(), "ZExt only operates on integer", &I);
546   Assert1(DestTy->isInteger(), "ZExt only produces an integer", &I);
547   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
548   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
549
550   Assert1(SrcBitSize < DestBitSize,"Type too small for ZExt", &I);
551
552   visitInstruction(I);
553 }
554
555 void Verifier::visitSExtInst(SExtInst &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   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
562   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
563
564   Assert1(SrcTy->isInteger(), "SExt only operates on integer", &I);
565   Assert1(DestTy->isInteger(), "SExt only produces an integer", &I);
566   Assert1(SrcBitSize < DestBitSize,"Type too small for SExt", &I);
567
568   visitInstruction(I);
569 }
570
571 void Verifier::visitFPTruncInst(FPTruncInst &I) {
572   // Get the source and destination types
573   const Type *SrcTy = I.getOperand(0)->getType();
574   const Type *DestTy = I.getType();
575   // Get the size of the types in bits, we'll need this later
576   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
577   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
578
579   Assert1(SrcTy->isFloatingPoint(),"FPTrunc only operates on FP", &I);
580   Assert1(DestTy->isFloatingPoint(),"FPTrunc only produces an FP", &I);
581   Assert1(SrcBitSize > DestBitSize,"DestTy too big for FPTrunc", &I);
582
583   visitInstruction(I);
584 }
585
586 void Verifier::visitFPExtInst(FPExtInst &I) {
587   // Get the source and destination types
588   const Type *SrcTy = I.getOperand(0)->getType();
589   const Type *DestTy = I.getType();
590
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(),"FPExt only operates on FP", &I);
596   Assert1(DestTy->isFloatingPoint(),"FPExt only produces an FP", &I);
597   Assert1(SrcBitSize < DestBitSize,"DestTy too small for FPExt", &I);
598
599   visitInstruction(I);
600 }
601
602 void Verifier::visitUIToFPInst(UIToFPInst &I) {
603   // Get the source and destination types
604   const Type *SrcTy = I.getOperand(0)->getType();
605   const Type *DestTy = I.getType();
606
607   Assert1(SrcTy->isInteger(),"UInt2FP source must be integral", &I);
608   Assert1(DestTy->isFloatingPoint(),"UInt2FP result must be FP", &I);
609
610   visitInstruction(I);
611 }
612
613 void Verifier::visitSIToFPInst(SIToFPInst &I) {
614   // Get the source and destination types
615   const Type *SrcTy = I.getOperand(0)->getType();
616   const Type *DestTy = I.getType();
617
618   Assert1(SrcTy->isInteger(),"SInt2FP source must be integral", &I);
619   Assert1(DestTy->isFloatingPoint(),"SInt2FP result must be FP", &I);
620
621   visitInstruction(I);
622 }
623
624 void Verifier::visitFPToUIInst(FPToUIInst &I) {
625   // Get the source and destination types
626   const Type *SrcTy = I.getOperand(0)->getType();
627   const Type *DestTy = I.getType();
628
629   Assert1(SrcTy->isFloatingPoint(),"FP2UInt source must be FP", &I);
630   Assert1(DestTy->isInteger(),"FP2UInt result must be integral", &I);
631
632   visitInstruction(I);
633 }
634
635 void Verifier::visitFPToSIInst(FPToSIInst &I) {
636   // Get the source and destination types
637   const Type *SrcTy = I.getOperand(0)->getType();
638   const Type *DestTy = I.getType();
639
640   Assert1(SrcTy->isFloatingPoint(),"FPToSI source must be FP", &I);
641   Assert1(DestTy->isInteger(),"FP2ToI result must be integral", &I);
642
643   visitInstruction(I);
644 }
645
646 void Verifier::visitPtrToIntInst(PtrToIntInst &I) {
647   // Get the source and destination types
648   const Type *SrcTy = I.getOperand(0)->getType();
649   const Type *DestTy = I.getType();
650
651   Assert1(isa<PointerType>(SrcTy), "PtrToInt source must be pointer", &I);
652   Assert1(DestTy->isInteger(), "PtrToInt result must be integral", &I);
653
654   visitInstruction(I);
655 }
656
657 void Verifier::visitIntToPtrInst(IntToPtrInst &I) {
658   // Get the source and destination types
659   const Type *SrcTy = I.getOperand(0)->getType();
660   const Type *DestTy = I.getType();
661
662   Assert1(SrcTy->isInteger(), "IntToPtr source must be an integral", &I);
663   Assert1(isa<PointerType>(DestTy), "IntToPtr result must be a pointer",&I);
664
665   visitInstruction(I);
666 }
667
668 void Verifier::visitBitCastInst(BitCastInst &I) {
669   // Get the source and destination types
670   const Type *SrcTy = I.getOperand(0)->getType();
671   const Type *DestTy = I.getType();
672
673   // Get the size of the types in bits, we'll need this later
674   unsigned SrcBitSize = SrcTy->getPrimitiveSizeInBits();
675   unsigned DestBitSize = DestTy->getPrimitiveSizeInBits();
676
677   // BitCast implies a no-op cast of type only. No bits change.
678   // However, you can't cast pointers to anything but pointers.
679   Assert1(isa<PointerType>(DestTy) == isa<PointerType>(DestTy),
680           "Bitcast requires both operands to be pointer or neither", &I);
681   Assert1(SrcBitSize == DestBitSize, "Bitcast requies types of same width", &I);
682
683   visitInstruction(I);
684 }
685
686 /// visitPHINode - Ensure that a PHI node is well formed.
687 ///
688 void Verifier::visitPHINode(PHINode &PN) {
689   // Ensure that the PHI nodes are all grouped together at the top of the block.
690   // This can be tested by checking whether the instruction before this is
691   // either nonexistent (because this is begin()) or is a PHI node.  If not,
692   // then there is some other instruction before a PHI.
693   Assert2(&PN == &PN.getParent()->front() || 
694           isa<PHINode>(--BasicBlock::iterator(&PN)),
695           "PHI nodes not grouped at top of basic block!",
696           &PN, PN.getParent());
697
698   // Check that all of the operands of the PHI node have the same type as the
699   // result.
700   for (unsigned i = 0, e = PN.getNumIncomingValues(); i != e; ++i)
701     Assert1(PN.getType() == PN.getIncomingValue(i)->getType(),
702             "PHI node operands are not the same type as the result!", &PN);
703
704   // All other PHI node constraints are checked in the visitBasicBlock method.
705
706   visitInstruction(PN);
707 }
708
709 void Verifier::visitCallInst(CallInst &CI) {
710   Assert1(isa<PointerType>(CI.getOperand(0)->getType()),
711           "Called function must be a pointer!", &CI);
712   const PointerType *FPTy = cast<PointerType>(CI.getOperand(0)->getType());
713   Assert1(isa<FunctionType>(FPTy->getElementType()),
714           "Called function is not pointer to function type!", &CI);
715
716   const FunctionType *FTy = cast<FunctionType>(FPTy->getElementType());
717
718   // Verify that the correct number of arguments are being passed
719   if (FTy->isVarArg())
720     Assert1(CI.getNumOperands()-1 >= FTy->getNumParams(),
721             "Called function requires more parameters than were provided!",&CI);
722   else
723     Assert1(CI.getNumOperands()-1 == FTy->getNumParams(),
724             "Incorrect number of arguments passed to called function!", &CI);
725
726   // Verify that all arguments to the call match the function type...
727   for (unsigned i = 0, e = FTy->getNumParams(); i != e; ++i)
728     Assert3(CI.getOperand(i+1)->getType() == FTy->getParamType(i),
729             "Call parameter type does not match function signature!",
730             CI.getOperand(i+1), FTy->getParamType(i), &CI);
731
732   if (Function *F = CI.getCalledFunction())
733     if (Intrinsic::ID ID = (Intrinsic::ID)F->getIntrinsicID())
734       visitIntrinsicFunctionCall(ID, CI);
735
736   visitInstruction(CI);
737 }
738
739 /// visitBinaryOperator - Check that both arguments to the binary operator are
740 /// of the same type!
741 ///
742 void Verifier::visitBinaryOperator(BinaryOperator &B) {
743   Assert1(B.getOperand(0)->getType() == B.getOperand(1)->getType(),
744           "Both operands to a binary operator are not of the same type!", &B);
745
746   switch (B.getOpcode()) {
747   // Check that logical operators are only used with integral operands.
748   case Instruction::And:
749   case Instruction::Or:
750   case Instruction::Xor:
751     Assert1(B.getType()->isInteger() ||
752             (isa<VectorType>(B.getType()) && 
753              cast<VectorType>(B.getType())->getElementType()->isInteger()),
754             "Logical operators only work with integral types!", &B);
755     Assert1(B.getType() == B.getOperand(0)->getType(),
756             "Logical operators must have same type for operands and result!",
757             &B);
758     break;
759   case Instruction::Shl:
760   case Instruction::LShr:
761   case Instruction::AShr:
762     Assert1(B.getType()->isInteger(),
763             "Shift must return an integer result!", &B);
764     Assert1(B.getType() == B.getOperand(0)->getType(),
765             "Shift return type must be same as operands!", &B);
766     /* FALL THROUGH */
767   default:
768     // Arithmetic operators only work on integer or fp values
769     Assert1(B.getType() == B.getOperand(0)->getType(),
770             "Arithmetic operators must have same type for operands and result!",
771             &B);
772     Assert1(B.getType()->isInteger() || B.getType()->isFloatingPoint() ||
773             isa<VectorType>(B.getType()),
774             "Arithmetic operators must have integer, fp, or vector type!", &B);
775     break;
776   }
777
778   visitInstruction(B);
779 }
780
781 void Verifier::visitICmpInst(ICmpInst& IC) {
782   // Check that the operands are the same type
783   const Type* Op0Ty = IC.getOperand(0)->getType();
784   const Type* Op1Ty = IC.getOperand(1)->getType();
785   Assert1(Op0Ty == Op1Ty,
786           "Both operands to ICmp instruction are not of the same type!", &IC);
787   // Check that the operands are the right type
788   Assert1(Op0Ty->isInteger() || isa<PointerType>(Op0Ty),
789           "Invalid operand types for ICmp instruction", &IC);
790   visitInstruction(IC);
791 }
792
793 void Verifier::visitFCmpInst(FCmpInst& FC) {
794   // Check that the operands are the same type
795   const Type* Op0Ty = FC.getOperand(0)->getType();
796   const Type* Op1Ty = FC.getOperand(1)->getType();
797   Assert1(Op0Ty == Op1Ty,
798           "Both operands to FCmp instruction are not of the same type!", &FC);
799   // Check that the operands are the right type
800   Assert1(Op0Ty->isFloatingPoint(),
801           "Invalid operand types for FCmp instruction", &FC);
802   visitInstruction(FC);
803 }
804
805 void Verifier::visitExtractElementInst(ExtractElementInst &EI) {
806   Assert1(ExtractElementInst::isValidOperands(EI.getOperand(0),
807                                               EI.getOperand(1)),
808           "Invalid extractelement operands!", &EI);
809   visitInstruction(EI);
810 }
811
812 void Verifier::visitInsertElementInst(InsertElementInst &IE) {
813   Assert1(InsertElementInst::isValidOperands(IE.getOperand(0),
814                                              IE.getOperand(1),
815                                              IE.getOperand(2)),
816           "Invalid insertelement operands!", &IE);
817   visitInstruction(IE);
818 }
819
820 void Verifier::visitShuffleVectorInst(ShuffleVectorInst &SV) {
821   Assert1(ShuffleVectorInst::isValidOperands(SV.getOperand(0), SV.getOperand(1),
822                                              SV.getOperand(2)),
823           "Invalid shufflevector operands!", &SV);
824   Assert1(SV.getType() == SV.getOperand(0)->getType(),
825           "Result of shufflevector must match first operand type!", &SV);
826   
827   // Check to see if Mask is valid.
828   if (const ConstantVector *MV = dyn_cast<ConstantVector>(SV.getOperand(2))) {
829     for (unsigned i = 0, e = MV->getNumOperands(); i != e; ++i) {
830       Assert1(isa<ConstantInt>(MV->getOperand(i)) ||
831               isa<UndefValue>(MV->getOperand(i)),
832               "Invalid shufflevector shuffle mask!", &SV);
833     }
834   } else {
835     Assert1(isa<UndefValue>(SV.getOperand(2)) || 
836             isa<ConstantAggregateZero>(SV.getOperand(2)),
837             "Invalid shufflevector shuffle mask!", &SV);
838   }
839   
840   visitInstruction(SV);
841 }
842
843 void Verifier::visitGetElementPtrInst(GetElementPtrInst &GEP) {
844   SmallVector<Value*, 16> Idxs(GEP.idx_begin(), GEP.idx_end());
845   const Type *ElTy =
846     GetElementPtrInst::getIndexedType(GEP.getOperand(0)->getType(),
847                                       &Idxs[0], Idxs.size(), true);
848   Assert1(ElTy, "Invalid indices for GEP pointer type!", &GEP);
849   Assert2(isa<PointerType>(GEP.getType()) &&
850           cast<PointerType>(GEP.getType())->getElementType() == ElTy,
851           "GEP is not of right type for indices!", &GEP, ElTy);
852   visitInstruction(GEP);
853 }
854
855 void Verifier::visitLoadInst(LoadInst &LI) {
856   const Type *ElTy =
857     cast<PointerType>(LI.getOperand(0)->getType())->getElementType();
858   Assert2(ElTy == LI.getType(),
859           "Load result type does not match pointer operand type!", &LI, ElTy);
860   visitInstruction(LI);
861 }
862
863 void Verifier::visitStoreInst(StoreInst &SI) {
864   const Type *ElTy =
865     cast<PointerType>(SI.getOperand(1)->getType())->getElementType();
866   Assert2(ElTy == SI.getOperand(0)->getType(),
867           "Stored value type does not match pointer operand type!", &SI, ElTy);
868   visitInstruction(SI);
869 }
870
871
872 /// verifyInstruction - Verify that an instruction is well formed.
873 ///
874 void Verifier::visitInstruction(Instruction &I) {
875   BasicBlock *BB = I.getParent();
876   Assert1(BB, "Instruction not embedded in basic block!", &I);
877
878   if (!isa<PHINode>(I)) {   // Check that non-phi nodes are not self referential
879     for (Value::use_iterator UI = I.use_begin(), UE = I.use_end();
880          UI != UE; ++UI)
881       Assert1(*UI != (User*)&I ||
882               !DT->dominates(&BB->getParent()->getEntryBlock(), BB),
883               "Only PHI nodes may reference their own value!", &I);
884   }
885
886   // Check that void typed values don't have names
887   Assert1(I.getType() != Type::VoidTy || !I.hasName(),
888           "Instruction has a name, but provides a void value!", &I);
889
890   // Check that the return value of the instruction is either void or a legal
891   // value type.
892   Assert1(I.getType() == Type::VoidTy || I.getType()->isFirstClassType(),
893           "Instruction returns a non-scalar type!", &I);
894
895   // Check that all uses of the instruction, if they are instructions
896   // themselves, actually have parent basic blocks.  If the use is not an
897   // instruction, it is an error!
898   for (User::use_iterator UI = I.use_begin(), UE = I.use_end();
899        UI != UE; ++UI) {
900     Assert1(isa<Instruction>(*UI), "Use of instruction is not an instruction!",
901             *UI);
902     Instruction *Used = cast<Instruction>(*UI);
903     Assert2(Used->getParent() != 0, "Instruction referencing instruction not"
904             " embeded in a basic block!", &I, Used);
905   }
906
907   for (unsigned i = 0, e = I.getNumOperands(); i != e; ++i) {
908     Assert1(I.getOperand(i) != 0, "Instruction has null operand!", &I);
909
910     // Check to make sure that only first-class-values are operands to
911     // instructions.
912     Assert1(I.getOperand(i)->getType()->isFirstClassType(),
913             "Instruction operands must be first-class values!", &I);
914   
915     if (Function *F = dyn_cast<Function>(I.getOperand(i))) {
916       // Check to make sure that the "address of" an intrinsic function is never
917       // taken.
918       Assert1(!F->isIntrinsic() || (i == 0 && isa<CallInst>(I)),
919               "Cannot take the address of an intrinsic!", &I);
920       Assert1(F->getParent() == Mod, "Referencing function in another module!",
921               &I);
922     } else if (BasicBlock *OpBB = dyn_cast<BasicBlock>(I.getOperand(i))) {
923       Assert1(OpBB->getParent() == BB->getParent(),
924               "Referring to a basic block in another function!", &I);
925     } else if (Argument *OpArg = dyn_cast<Argument>(I.getOperand(i))) {
926       Assert1(OpArg->getParent() == BB->getParent(),
927               "Referring to an argument in another function!", &I);
928     } else if (GlobalValue *GV = dyn_cast<GlobalValue>(I.getOperand(i))) {
929       Assert1(GV->getParent() == Mod, "Referencing global in another module!",
930               &I);
931     } else if (Instruction *Op = dyn_cast<Instruction>(I.getOperand(i))) {
932       BasicBlock *OpBlock = Op->getParent();
933
934       // Check that a definition dominates all of its uses.
935       if (!isa<PHINode>(I)) {
936         // Invoke results are only usable in the normal destination, not in the
937         // exceptional destination.
938         if (InvokeInst *II = dyn_cast<InvokeInst>(Op)) {
939           OpBlock = II->getNormalDest();
940           
941           Assert2(OpBlock != II->getUnwindDest(),
942                   "No uses of invoke possible due to dominance structure!",
943                   Op, II);
944           
945           // If the normal successor of an invoke instruction has multiple
946           // predecessors, then the normal edge from the invoke is critical, so
947           // the invoke value can only be live if the destination block
948           // dominates all of it's predecessors (other than the invoke) or if
949           // the invoke value is only used by a phi in the successor.
950           if (!OpBlock->getSinglePredecessor() &&
951               DT->dominates(&BB->getParent()->getEntryBlock(), BB)) {
952             // The first case we allow is if the use is a PHI operand in the
953             // normal block, and if that PHI operand corresponds to the invoke's
954             // block.
955             bool Bad = true;
956             if (PHINode *PN = dyn_cast<PHINode>(&I))
957               if (PN->getParent() == OpBlock &&
958                   PN->getIncomingBlock(i/2) == Op->getParent())
959                 Bad = false;
960             
961             // If it is used by something non-phi, then the other case is that
962             // 'OpBlock' dominates all of its predecessors other than the
963             // invoke.  In this case, the invoke value can still be used.
964             if (Bad) {
965               Bad = false;
966               for (pred_iterator PI = pred_begin(OpBlock),
967                    E = pred_end(OpBlock); PI != E; ++PI) {
968                 if (*PI != II->getParent() && !DT->dominates(OpBlock, *PI)) {
969                   Bad = true;
970                   break;
971                 }
972               }
973             }
974             Assert2(!Bad,
975                     "Invoke value defined on critical edge but not dead!", &I,
976                     Op);
977           }
978         } else if (OpBlock == BB) {
979           // If they are in the same basic block, make sure that the definition
980           // comes before the use.
981           Assert2(InstsInThisBlock.count(Op) ||
982                   !DT->dominates(&BB->getParent()->getEntryBlock(), BB),
983                   "Instruction does not dominate all uses!", Op, &I);
984         }
985
986         // Definition must dominate use unless use is unreachable!
987         Assert2(DT->dominates(OpBlock, BB) ||
988                 !DT->dominates(&BB->getParent()->getEntryBlock(), BB),
989                 "Instruction does not dominate all uses!", Op, &I);
990       } else {
991         // PHI nodes are more difficult than other nodes because they actually
992         // "use" the value in the predecessor basic blocks they correspond to.
993         BasicBlock *PredBB = cast<BasicBlock>(I.getOperand(i+1));
994         Assert2(DT->dominates(OpBlock, PredBB) ||
995                 !DT->dominates(&BB->getParent()->getEntryBlock(), PredBB),
996                 "Instruction does not dominate all uses!", Op, &I);
997       }
998     } else if (isa<InlineAsm>(I.getOperand(i))) {
999       Assert1(i == 0 && isa<CallInst>(I),
1000               "Cannot take the address of an inline asm!", &I);
1001     }
1002   }
1003   InstsInThisBlock.insert(&I);
1004 }
1005
1006 /// visitIntrinsicFunction - Allow intrinsics to be verified in different ways.
1007 ///
1008 void Verifier::visitIntrinsicFunctionCall(Intrinsic::ID ID, CallInst &CI) {
1009   Function *IF = CI.getCalledFunction();
1010   Assert1(IF->isDeclaration(), "Intrinsic functions should never be defined!",
1011           IF);
1012   
1013 #define GET_INTRINSIC_VERIFIER
1014 #include "llvm/Intrinsics.gen"
1015 #undef GET_INTRINSIC_VERIFIER
1016 }
1017
1018 /// VerifyIntrinsicPrototype - TableGen emits calls to this function into
1019 /// Intrinsics.gen.  This implements a little state machine that verifies the
1020 /// prototype of intrinsics.
1021 void Verifier::VerifyIntrinsicPrototype(Intrinsic::ID ID, Function *F, ...) {
1022   va_list VA;
1023   va_start(VA, F);
1024   
1025   const FunctionType *FTy = F->getFunctionType();
1026   
1027   // For overloaded intrinsics, the Suffix of the function name must match the
1028   // types of the arguments. This variable keeps track of the expected
1029   // suffix, to be checked at the end.
1030   std::string Suffix;
1031
1032   // Note that "arg#0" is the return type.
1033   for (unsigned ArgNo = 0; 1; ++ArgNo) {
1034     int TypeID = va_arg(VA, int);
1035
1036     if (TypeID == -2) {
1037       break;
1038     }
1039
1040     if (TypeID == -1) {
1041       if (ArgNo != FTy->getNumParams()+1)
1042         CheckFailed("Intrinsic prototype has too many arguments!", F);
1043       break;
1044     }
1045
1046     if (ArgNo == FTy->getNumParams()+1) {
1047       CheckFailed("Intrinsic prototype has too few arguments!", F);
1048       break;
1049     }
1050     
1051     const Type *Ty;
1052     if (ArgNo == 0)
1053       Ty = FTy->getReturnType();
1054     else
1055       Ty = FTy->getParamType(ArgNo-1);
1056     
1057     if (TypeID != Ty->getTypeID()) {
1058       if (ArgNo == 0)
1059         CheckFailed("Intrinsic prototype has incorrect result type!", F);
1060       else
1061         CheckFailed("Intrinsic parameter #" + utostr(ArgNo-1) + " is wrong!",F);
1062       break;
1063     }
1064
1065     if (TypeID == Type::IntegerTyID) {
1066       unsigned ExpectedBits = (unsigned) va_arg(VA, int);
1067       unsigned GotBits = cast<IntegerType>(Ty)->getBitWidth();
1068       if (ExpectedBits == 0) {
1069         Suffix += ".i" + utostr(GotBits);
1070       } else if (GotBits != ExpectedBits) {
1071         std::string bitmsg = " Expected " + utostr(ExpectedBits) + " but got "+
1072                              utostr(GotBits) + " bits.";
1073         if (ArgNo == 0)
1074           CheckFailed("Intrinsic prototype has incorrect integer result width!"
1075                       + bitmsg, F);
1076         else
1077           CheckFailed("Intrinsic parameter #" + utostr(ArgNo-1) + " has "
1078                       "incorrect integer width!" + bitmsg, F);
1079         break;
1080       }
1081       // Check some constraints on various intrinsics.
1082       switch (ID) {
1083         default: break; // Not everything needs to be checked.
1084         case Intrinsic::bswap:
1085           if (GotBits < 16 || GotBits % 16 != 0)
1086             CheckFailed("Intrinsic requires even byte width argument", F);
1087           /* FALL THROUGH */
1088         case Intrinsic::part_set:
1089         case Intrinsic::part_select:
1090           if (ArgNo == 1) {
1091             unsigned ResultBits = 
1092               cast<IntegerType>(FTy->getReturnType())->getBitWidth();
1093             if (GotBits != ResultBits)
1094               CheckFailed("Intrinsic requires the bit widths of the first "
1095                           "parameter and the result to match", F);
1096           }
1097           break;
1098       }
1099     } else if (TypeID == Type::VectorTyID) {
1100       // If this is a packed argument, verify the number and type of elements.
1101       const VectorType *PTy = cast<VectorType>(Ty);
1102       int ElemTy = va_arg(VA, int);
1103       if (ElemTy != PTy->getElementType()->getTypeID()) {
1104         CheckFailed("Intrinsic prototype has incorrect vector element type!",
1105                     F);
1106         break;
1107       }
1108       if (ElemTy == Type::IntegerTyID) {
1109         unsigned NumBits = (unsigned)va_arg(VA, int);
1110         unsigned ExpectedBits = 
1111           cast<IntegerType>(PTy->getElementType())->getBitWidth();
1112         if (NumBits != ExpectedBits) {
1113           CheckFailed("Intrinsic prototype has incorrect vector element type!",
1114                       F);
1115           break;
1116         }
1117       }
1118       if ((unsigned)va_arg(VA, int) != PTy->getNumElements()) {
1119         CheckFailed("Intrinsic prototype has incorrect number of "
1120                     "vector elements!",F);
1121           break;
1122       }
1123     }
1124   }
1125
1126   va_end(VA);
1127
1128   // If we computed a Suffix then the intrinsic is overloaded and we need to 
1129   // make sure that the name of the function is correct. We add the suffix to
1130   // the name of the intrinsic and compare against the given function name. If
1131   // they are not the same, the function name is invalid. This ensures that
1132   // overloading of intrinsics uses a sane and consistent naming convention.
1133   if (!Suffix.empty()) {
1134     std::string Name(Intrinsic::getName(ID));
1135     if (Name + Suffix != F->getName())
1136       CheckFailed("Overloaded intrinsic has incorrect suffix: '" +
1137                   F->getName().substr(Name.length()) + "'. It should be '" +
1138                   Suffix + "'", F);
1139   }
1140 }
1141
1142
1143 //===----------------------------------------------------------------------===//
1144 //  Implement the public interfaces to this file...
1145 //===----------------------------------------------------------------------===//
1146
1147 FunctionPass *llvm::createVerifierPass(VerifierFailureAction action) {
1148   return new Verifier(action);
1149 }
1150
1151
1152 // verifyFunction - Create
1153 bool llvm::verifyFunction(const Function &f, VerifierFailureAction action) {
1154   Function &F = const_cast<Function&>(f);
1155   assert(!F.isDeclaration() && "Cannot verify external functions");
1156
1157   FunctionPassManager FPM(new ExistingModuleProvider(F.getParent()));
1158   Verifier *V = new Verifier(action);
1159   FPM.add(V);
1160   FPM.run(F);
1161   return V->Broken;
1162 }
1163
1164 /// verifyModule - Check a module for errors, printing messages on stderr.
1165 /// Return true if the module is corrupt.
1166 ///
1167 bool llvm::verifyModule(const Module &M, VerifierFailureAction action,
1168                         std::string *ErrorInfo) {
1169   PassManager PM;
1170   Verifier *V = new Verifier(action);
1171   PM.add(V);
1172   PM.run((Module&)M);
1173   
1174   if (ErrorInfo && V->Broken)
1175     *ErrorInfo = V->msgs.str();
1176   return V->Broken;
1177 }
1178
1179 // vim: sw=2