1 //===-- DifferenceEngine.cpp - Structural function/module comparison ------===//
2 //
3 //                     The LLVM Compiler Infrastructure
4 //
5 // This file is distributed under the University of Illinois Open Source
6 // License. See LICENSE.TXT for details.
7 //
8 //===----------------------------------------------------------------------===//
9 //
10 // This header defines the implementation of the LLVM difference
11 // engine, which structurally compares global values within a module.
12 //
13 //===----------------------------------------------------------------------===//
14 
15 #include "DifferenceEngine.h"
16 #include "llvm/ADT/DenseMap.h"
17 #include "llvm/ADT/DenseSet.h"
18 #include "llvm/ADT/SmallVector.h"
19 #include "llvm/ADT/StringRef.h"
20 #include "llvm/ADT/StringSet.h"
21 #include "llvm/IR/CFG.h"
22 #include "llvm/IR/CallSite.h"
23 #include "llvm/IR/Constants.h"
24 #include "llvm/IR/Function.h"
25 #include "llvm/IR/Instructions.h"
26 #include "llvm/IR/Module.h"
27 #include "llvm/Support/ErrorHandling.h"
28 #include "llvm/Support/raw_ostream.h"
29 #include "llvm/Support/type_traits.h"
30 #include <utility>
31 
32 using namespace llvm;
33 
34 namespace {
35 
36 /// A priority queue, implemented as a heap.
37 template <class T, class Sorter, unsigned InlineCapacity>
38 class PriorityQueue {
39   Sorter Precedes;
40   llvm::SmallVector<T, InlineCapacity> Storage;
41 
42 public:
PriorityQueue(const Sorter & Precedes)43   PriorityQueue(const Sorter &Precedes) : Precedes(Precedes) {}
44 
45   /// Checks whether the heap is empty.
empty() const46   bool empty() const { return Storage.empty(); }
47 
48   /// Insert a new value on the heap.
insert(const T & V)49   void insert(const T &V) {
50     unsigned Index = Storage.size();
51     Storage.push_back(V);
52     if (Index == 0) return;
53 
54     T *data = Storage.data();
55     while (true) {
56       unsigned Target = (Index + 1) / 2 - 1;
57       if (!Precedes(data[Index], data[Target])) return;
58       std::swap(data[Index], data[Target]);
59       if (Target == 0) return;
60       Index = Target;
61     }
62   }
63 
64   /// Remove the minimum value in the heap.  Only valid on a non-empty heap.
remove_min()65   T remove_min() {
66     assert(!empty());
67     T tmp = Storage[0];
68 
69     unsigned NewSize = Storage.size() - 1;
70     if (NewSize) {
71       // Move the slot at the end to the beginning.
72       if (isPodLike<T>::value)
73         Storage[0] = Storage[NewSize];
74       else
75         std::swap(Storage[0], Storage[NewSize]);
76 
77       // Bubble the root up as necessary.
78       unsigned Index = 0;
79       while (true) {
80         // With a 1-based index, the children would be Index*2 and Index*2+1.
81         unsigned R = (Index + 1) * 2;
82         unsigned L = R - 1;
83 
84         // If R is out of bounds, we're done after this in any case.
85         if (R >= NewSize) {
86           // If L is also out of bounds, we're done immediately.
87           if (L >= NewSize) break;
88 
89           // Otherwise, test whether we should swap L and Index.
90           if (Precedes(Storage[L], Storage[Index]))
91             std::swap(Storage[L], Storage[Index]);
92           break;
93         }
94 
95         // Otherwise, we need to compare with the smaller of L and R.
96         // Prefer R because it's closer to the end of the array.
97         unsigned IndexToTest = (Precedes(Storage[L], Storage[R]) ? L : R);
98 
99         // If Index is >= the min of L and R, then heap ordering is restored.
100         if (!Precedes(Storage[IndexToTest], Storage[Index]))
101           break;
102 
103         // Otherwise, keep bubbling up.
104         std::swap(Storage[IndexToTest], Storage[Index]);
105         Index = IndexToTest;
106       }
107     }
108     Storage.pop_back();
109 
110     return tmp;
111   }
112 };
113 
114 /// A function-scope difference engine.
115 class FunctionDifferenceEngine {
116   DifferenceEngine &Engine;
117 
118   /// The current mapping from old local values to new local values.
119   DenseMap<Value*, Value*> Values;
120 
121   /// The current mapping from old blocks to new blocks.
122   DenseMap<BasicBlock*, BasicBlock*> Blocks;
123 
124   DenseSet<std::pair<Value*, Value*> > TentativeValues;
125 
getUnprocPredCount(BasicBlock * Block) const126   unsigned getUnprocPredCount(BasicBlock *Block) const {
127     unsigned Count = 0;
128     for (pred_iterator I = pred_begin(Block), E = pred_end(Block); I != E; ++I)
129       if (!Blocks.count(*I)) Count++;
130     return Count;
131   }
132 
133   typedef std::pair<BasicBlock*, BasicBlock*> BlockPair;
134 
135   /// A type which sorts a priority queue by the number of unprocessed
136   /// predecessor blocks it has remaining.
137   ///
138   /// This is actually really expensive to calculate.
139   struct QueueSorter {
140     const FunctionDifferenceEngine &fde;
QueueSorter__anon6bb4de1d0111::FunctionDifferenceEngine::QueueSorter141     explicit QueueSorter(const FunctionDifferenceEngine &fde) : fde(fde) {}
142 
operator ()__anon6bb4de1d0111::FunctionDifferenceEngine::QueueSorter143     bool operator()(const BlockPair &Old, const BlockPair &New) {
144       return fde.getUnprocPredCount(Old.first)
145            < fde.getUnprocPredCount(New.first);
146     }
147   };
148 
149   /// A queue of unified blocks to process.
150   PriorityQueue<BlockPair, QueueSorter, 20> Queue;
151 
152   /// Try to unify the given two blocks.  Enqueues them for processing
153   /// if they haven't already been processed.
154   ///
155   /// Returns true if there was a problem unifying them.
tryUnify(BasicBlock * L,BasicBlock * R)156   bool tryUnify(BasicBlock *L, BasicBlock *R) {
157     BasicBlock *&Ref = Blocks[L];
158 
159     if (Ref) {
160       if (Ref == R) return false;
161 
162       Engine.logf("successor %l cannot be equivalent to %r; "
163                   "it's already equivalent to %r")
164         << L << R << Ref;
165       return true;
166     }
167 
168     Ref = R;
169     Queue.insert(BlockPair(L, R));
170     return false;
171   }
172 
173   /// Unifies two instructions, given that they're known not to have
174   /// structural differences.
unify(Instruction * L,Instruction * R)175   void unify(Instruction *L, Instruction *R) {
176     DifferenceEngine::Context C(Engine, L, R);
177 
178     bool Result = diff(L, R, true, true);
179     assert(!Result && "structural differences second time around?");
180     (void) Result;
181     if (!L->use_empty())
182       Values[L] = R;
183   }
184 
processQueue()185   void processQueue() {
186     while (!Queue.empty()) {
187       BlockPair Pair = Queue.remove_min();
188       diff(Pair.first, Pair.second);
189     }
190   }
191 
diff(BasicBlock * L,BasicBlock * R)192   void diff(BasicBlock *L, BasicBlock *R) {
193     DifferenceEngine::Context C(Engine, L, R);
194 
195     BasicBlock::iterator LI = L->begin(), LE = L->end();
196     BasicBlock::iterator RI = R->begin();
197 
198     do {
199       assert(LI != LE && RI != R->end());
200       Instruction *LeftI = &*LI, *RightI = &*RI;
201 
202       // If the instructions differ, start the more sophisticated diff
203       // algorithm at the start of the block.
204       if (diff(LeftI, RightI, false, false)) {
205         TentativeValues.clear();
206         return runBlockDiff(L->begin(), R->begin());
207       }
208 
209       // Otherwise, tentatively unify them.
210       if (!LeftI->use_empty())
211         TentativeValues.insert(std::make_pair(LeftI, RightI));
212 
213       ++LI, ++RI;
214     } while (LI != LE); // This is sufficient: we can't get equality of
215                         // terminators if there are residual instructions.
216 
217     // Unify everything in the block, non-tentatively this time.
218     TentativeValues.clear();
219     for (LI = L->begin(), RI = R->begin(); LI != LE; ++LI, ++RI)
220       unify(&*LI, &*RI);
221   }
222 
223   bool matchForBlockDiff(Instruction *L, Instruction *R);
224   void runBlockDiff(BasicBlock::iterator LI, BasicBlock::iterator RI);
225 
diffCallSites(CallSite L,CallSite R,bool Complain)226   bool diffCallSites(CallSite L, CallSite R, bool Complain) {
227     // FIXME: call attributes
228     if (!equivalentAsOperands(L.getCalledValue(), R.getCalledValue())) {
229       if (Complain) Engine.log("called functions differ");
230       return true;
231     }
232     if (L.arg_size() != R.arg_size()) {
233       if (Complain) Engine.log("argument counts differ");
234       return true;
235     }
236     for (unsigned I = 0, E = L.arg_size(); I != E; ++I)
237       if (!equivalentAsOperands(L.getArgument(I), R.getArgument(I))) {
238         if (Complain)
239           Engine.logf("arguments %l and %r differ")
240             << L.getArgument(I) << R.getArgument(I);
241         return true;
242       }
243     return false;
244   }
245 
diff(Instruction * L,Instruction * R,bool Complain,bool TryUnify)246   bool diff(Instruction *L, Instruction *R, bool Complain, bool TryUnify) {
247     // FIXME: metadata (if Complain is set)
248 
249     // Different opcodes always imply different operations.
250     if (L->getOpcode() != R->getOpcode()) {
251       if (Complain) Engine.log("different instruction types");
252       return true;
253     }
254 
255     if (isa<CmpInst>(L)) {
256       if (cast<CmpInst>(L)->getPredicate()
257             != cast<CmpInst>(R)->getPredicate()) {
258         if (Complain) Engine.log("different predicates");
259         return true;
260       }
261     } else if (isa<CallInst>(L)) {
262       return diffCallSites(CallSite(L), CallSite(R), Complain);
263     } else if (isa<PHINode>(L)) {
264       // FIXME: implement.
265 
266       // This is really weird;  type uniquing is broken?
267       if (L->getType() != R->getType()) {
268         if (!L->getType()->isPointerTy() || !R->getType()->isPointerTy()) {
269           if (Complain) Engine.log("different phi types");
270           return true;
271         }
272       }
273       return false;
274 
275     // Terminators.
276     } else if (isa<InvokeInst>(L)) {
277       InvokeInst *LI = cast<InvokeInst>(L);
278       InvokeInst *RI = cast<InvokeInst>(R);
279       if (diffCallSites(CallSite(LI), CallSite(RI), Complain))
280         return true;
281 
282       if (TryUnify) {
283         tryUnify(LI->getNormalDest(), RI->getNormalDest());
284         tryUnify(LI->getUnwindDest(), RI->getUnwindDest());
285       }
286       return false;
287 
288     } else if (isa<BranchInst>(L)) {
289       BranchInst *LI = cast<BranchInst>(L);
290       BranchInst *RI = cast<BranchInst>(R);
291       if (LI->isConditional() != RI->isConditional()) {
292         if (Complain) Engine.log("branch conditionality differs");
293         return true;
294       }
295 
296       if (LI->isConditional()) {
297         if (!equivalentAsOperands(LI->getCondition(), RI->getCondition())) {
298           if (Complain) Engine.log("branch conditions differ");
299           return true;
300         }
301         if (TryUnify) tryUnify(LI->getSuccessor(1), RI->getSuccessor(1));
302       }
303       if (TryUnify) tryUnify(LI->getSuccessor(0), RI->getSuccessor(0));
304       return false;
305 
306     } else if (isa<SwitchInst>(L)) {
307       SwitchInst *LI = cast<SwitchInst>(L);
308       SwitchInst *RI = cast<SwitchInst>(R);
309       if (!equivalentAsOperands(LI->getCondition(), RI->getCondition())) {
310         if (Complain) Engine.log("switch conditions differ");
311         return true;
312       }
313       if (TryUnify) tryUnify(LI->getDefaultDest(), RI->getDefaultDest());
314 
315       bool Difference = false;
316 
317       DenseMap<ConstantInt*,BasicBlock*> LCases;
318 
319       for (SwitchInst::CaseIt I = LI->case_begin(), E = LI->case_end();
320            I != E; ++I)
321         LCases[I.getCaseValue()] = I.getCaseSuccessor();
322 
323       for (SwitchInst::CaseIt I = RI->case_begin(), E = RI->case_end();
324            I != E; ++I) {
325         ConstantInt *CaseValue = I.getCaseValue();
326         BasicBlock *LCase = LCases[CaseValue];
327         if (LCase) {
328           if (TryUnify) tryUnify(LCase, I.getCaseSuccessor());
329           LCases.erase(CaseValue);
330         } else if (Complain || !Difference) {
331           if (Complain)
332             Engine.logf("right switch has extra case %r") << CaseValue;
333           Difference = true;
334         }
335       }
336       if (!Difference)
337         for (DenseMap<ConstantInt*,BasicBlock*>::iterator
338                I = LCases.begin(), E = LCases.end(); I != E; ++I) {
339           if (Complain)
340             Engine.logf("left switch has extra case %l") << I->first;
341           Difference = true;
342         }
343       return Difference;
344     } else if (isa<UnreachableInst>(L)) {
345       return false;
346     }
347 
348     if (L->getNumOperands() != R->getNumOperands()) {
349       if (Complain) Engine.log("instructions have different operand counts");
350       return true;
351     }
352 
353     for (unsigned I = 0, E = L->getNumOperands(); I != E; ++I) {
354       Value *LO = L->getOperand(I), *RO = R->getOperand(I);
355       if (!equivalentAsOperands(LO, RO)) {
356         if (Complain) Engine.logf("operands %l and %r differ") << LO << RO;
357         return true;
358       }
359     }
360 
361     return false;
362   }
363 
equivalentAsOperands(Constant * L,Constant * R)364   bool equivalentAsOperands(Constant *L, Constant *R) {
365     // Use equality as a preliminary filter.
366     if (L == R)
367       return true;
368 
369     if (L->getValueID() != R->getValueID())
370       return false;
371 
372     // Ask the engine about global values.
373     if (isa<GlobalValue>(L))
374       return Engine.equivalentAsOperands(cast<GlobalValue>(L),
375                                          cast<GlobalValue>(R));
376 
377     // Compare constant expressions structurally.
378     if (isa<ConstantExpr>(L))
379       return equivalentAsOperands(cast<ConstantExpr>(L),
380                                   cast<ConstantExpr>(R));
381 
382     // Nulls of the "same type" don't always actually have the same
383     // type; I don't know why.  Just white-list them.
384     if (isa<ConstantPointerNull>(L))
385       return true;
386 
387     // Block addresses only match if we've already encountered the
388     // block.  FIXME: tentative matches?
389     if (isa<BlockAddress>(L))
390       return Blocks[cast<BlockAddress>(L)->getBasicBlock()]
391                  == cast<BlockAddress>(R)->getBasicBlock();
392 
393     return false;
394   }
395 
equivalentAsOperands(ConstantExpr * L,ConstantExpr * R)396   bool equivalentAsOperands(ConstantExpr *L, ConstantExpr *R) {
397     if (L == R)
398       return true;
399     if (L->getOpcode() != R->getOpcode())
400       return false;
401 
402     switch (L->getOpcode()) {
403     case Instruction::ICmp:
404     case Instruction::FCmp:
405       if (L->getPredicate() != R->getPredicate())
406         return false;
407       break;
408 
409     case Instruction::GetElementPtr:
410       // FIXME: inbounds?
411       break;
412 
413     default:
414       break;
415     }
416 
417     if (L->getNumOperands() != R->getNumOperands())
418       return false;
419 
420     for (unsigned I = 0, E = L->getNumOperands(); I != E; ++I)
421       if (!equivalentAsOperands(L->getOperand(I), R->getOperand(I)))
422         return false;
423 
424     return true;
425   }
426 
equivalentAsOperands(Value * L,Value * R)427   bool equivalentAsOperands(Value *L, Value *R) {
428     // Fall out if the values have different kind.
429     // This possibly shouldn't take priority over oracles.
430     if (L->getValueID() != R->getValueID())
431       return false;
432 
433     // Value subtypes:  Argument, Constant, Instruction, BasicBlock,
434     //                  InlineAsm, MDNode, MDString, PseudoSourceValue
435 
436     if (isa<Constant>(L))
437       return equivalentAsOperands(cast<Constant>(L), cast<Constant>(R));
438 
439     if (isa<Instruction>(L))
440       return Values[L] == R || TentativeValues.count(std::make_pair(L, R));
441 
442     if (isa<Argument>(L))
443       return Values[L] == R;
444 
445     if (isa<BasicBlock>(L))
446       return Blocks[cast<BasicBlock>(L)] != R;
447 
448     // Pretend everything else is identical.
449     return true;
450   }
451 
452   // Avoid a gcc warning about accessing 'this' in an initializer.
this_()453   FunctionDifferenceEngine *this_() { return this; }
454 
455 public:
FunctionDifferenceEngine(DifferenceEngine & Engine)456   FunctionDifferenceEngine(DifferenceEngine &Engine) :
457     Engine(Engine), Queue(QueueSorter(*this_())) {}
458 
diff(Function * L,Function * R)459   void diff(Function *L, Function *R) {
460     if (L->arg_size() != R->arg_size())
461       Engine.log("different argument counts");
462 
463     // Map the arguments.
464     for (Function::arg_iterator
465            LI = L->arg_begin(), LE = L->arg_end(),
466            RI = R->arg_begin(), RE = R->arg_end();
467          LI != LE && RI != RE; ++LI, ++RI)
468       Values[&*LI] = &*RI;
469 
470     tryUnify(&*L->begin(), &*R->begin());
471     processQueue();
472   }
473 };
474 
475 struct DiffEntry {
DiffEntry__anon6bb4de1d0111::DiffEntry476   DiffEntry() : Cost(0) {}
477 
478   unsigned Cost;
479   llvm::SmallVector<char, 8> Path; // actually of DifferenceEngine::DiffChange
480 };
481 
matchForBlockDiff(Instruction * L,Instruction * R)482 bool FunctionDifferenceEngine::matchForBlockDiff(Instruction *L,
483                                                  Instruction *R) {
484   return !diff(L, R, false, false);
485 }
486 
runBlockDiff(BasicBlock::iterator LStart,BasicBlock::iterator RStart)487 void FunctionDifferenceEngine::runBlockDiff(BasicBlock::iterator LStart,
488                                             BasicBlock::iterator RStart) {
489   BasicBlock::iterator LE = LStart->getParent()->end();
490   BasicBlock::iterator RE = RStart->getParent()->end();
491 
492   unsigned NL = std::distance(LStart, LE);
493 
494   SmallVector<DiffEntry, 20> Paths1(NL+1);
495   SmallVector<DiffEntry, 20> Paths2(NL+1);
496 
497   DiffEntry *Cur = Paths1.data();
498   DiffEntry *Next = Paths2.data();
499 
500   const unsigned LeftCost = 2;
501   const unsigned RightCost = 2;
502   const unsigned MatchCost = 0;
503 
504   assert(TentativeValues.empty());
505 
506   // Initialize the first column.
507   for (unsigned I = 0; I != NL+1; ++I) {
508     Cur[I].Cost = I * LeftCost;
509     for (unsigned J = 0; J != I; ++J)
510       Cur[I].Path.push_back(DC_left);
511   }
512 
513   for (BasicBlock::iterator RI = RStart; RI != RE; ++RI) {
514     // Initialize the first row.
515     Next[0] = Cur[0];
516     Next[0].Cost += RightCost;
517     Next[0].Path.push_back(DC_right);
518 
519     unsigned Index = 1;
520     for (BasicBlock::iterator LI = LStart; LI != LE; ++LI, ++Index) {
521       if (matchForBlockDiff(&*LI, &*RI)) {
522         Next[Index] = Cur[Index-1];
523         Next[Index].Cost += MatchCost;
524         Next[Index].Path.push_back(DC_match);
525         TentativeValues.insert(std::make_pair(&*LI, &*RI));
526       } else if (Next[Index-1].Cost <= Cur[Index].Cost) {
527         Next[Index] = Next[Index-1];
528         Next[Index].Cost += LeftCost;
529         Next[Index].Path.push_back(DC_left);
530       } else {
531         Next[Index] = Cur[Index];
532         Next[Index].Cost += RightCost;
533         Next[Index].Path.push_back(DC_right);
534       }
535     }
536 
537     std::swap(Cur, Next);
538   }
539 
540   // We don't need the tentative values anymore; everything from here
541   // on out should be non-tentative.
542   TentativeValues.clear();
543 
544   SmallVectorImpl<char> &Path = Cur[NL].Path;
545   BasicBlock::iterator LI = LStart, RI = RStart;
546 
547   DiffLogBuilder Diff(Engine.getConsumer());
548 
549   // Drop trailing matches.
550   while (Path.back() == DC_match)
551     Path.pop_back();
552 
553   // Skip leading matches.
554   SmallVectorImpl<char>::iterator
555     PI = Path.begin(), PE = Path.end();
556   while (PI != PE && *PI == DC_match) {
557     unify(&*LI, &*RI);
558     ++PI, ++LI, ++RI;
559   }
560 
561   for (; PI != PE; ++PI) {
562     switch (static_cast<DiffChange>(*PI)) {
563     case DC_match:
564       assert(LI != LE && RI != RE);
565       {
566         Instruction *L = &*LI, *R = &*RI;
567         unify(L, R);
568         Diff.addMatch(L, R);
569       }
570       ++LI; ++RI;
571       break;
572 
573     case DC_left:
574       assert(LI != LE);
575       Diff.addLeft(&*LI);
576       ++LI;
577       break;
578 
579     case DC_right:
580       assert(RI != RE);
581       Diff.addRight(&*RI);
582       ++RI;
583       break;
584     }
585   }
586 
587   // Finishing unifying and complaining about the tails of the block,
588   // which should be matches all the way through.
589   while (LI != LE) {
590     assert(RI != RE);
591     unify(&*LI, &*RI);
592     ++LI, ++RI;
593   }
594 
595   // If the terminators have different kinds, but one is an invoke and the
596   // other is an unconditional branch immediately following a call, unify
597   // the results and the destinations.
598   TerminatorInst *LTerm = LStart->getParent()->getTerminator();
599   TerminatorInst *RTerm = RStart->getParent()->getTerminator();
600   if (isa<BranchInst>(LTerm) && isa<InvokeInst>(RTerm)) {
601     if (cast<BranchInst>(LTerm)->isConditional()) return;
602     BasicBlock::iterator I = LTerm->getIterator();
603     if (I == LStart->getParent()->begin()) return;
604     --I;
605     if (!isa<CallInst>(*I)) return;
606     CallInst *LCall = cast<CallInst>(&*I);
607     InvokeInst *RInvoke = cast<InvokeInst>(RTerm);
608     if (!equivalentAsOperands(LCall->getCalledValue(), RInvoke->getCalledValue()))
609       return;
610     if (!LCall->use_empty())
611       Values[LCall] = RInvoke;
612     tryUnify(LTerm->getSuccessor(0), RInvoke->getNormalDest());
613   } else if (isa<InvokeInst>(LTerm) && isa<BranchInst>(RTerm)) {
614     if (cast<BranchInst>(RTerm)->isConditional()) return;
615     BasicBlock::iterator I = RTerm->getIterator();
616     if (I == RStart->getParent()->begin()) return;
617     --I;
618     if (!isa<CallInst>(*I)) return;
619     CallInst *RCall = cast<CallInst>(I);
620     InvokeInst *LInvoke = cast<InvokeInst>(LTerm);
621     if (!equivalentAsOperands(LInvoke->getCalledValue(), RCall->getCalledValue()))
622       return;
623     if (!LInvoke->use_empty())
624       Values[LInvoke] = RCall;
625     tryUnify(LInvoke->getNormalDest(), RTerm->getSuccessor(0));
626   }
627 }
628 
629 }
630 
anchor()631 void DifferenceEngine::Oracle::anchor() { }
632 
diff(Function * L,Function * R)633 void DifferenceEngine::diff(Function *L, Function *R) {
634   Context C(*this, L, R);
635 
636   // FIXME: types
637   // FIXME: attributes and CC
638   // FIXME: parameter attributes
639 
640   // If both are declarations, we're done.
641   if (L->empty() && R->empty())
642     return;
643   else if (L->empty())
644     log("left function is declaration, right function is definition");
645   else if (R->empty())
646     log("right function is declaration, left function is definition");
647   else
648     FunctionDifferenceEngine(*this).diff(L, R);
649 }
650 
diff(Module * L,Module * R)651 void DifferenceEngine::diff(Module *L, Module *R) {
652   StringSet<> LNames;
653   SmallVector<std::pair<Function*,Function*>, 20> Queue;
654 
655   for (Module::iterator I = L->begin(), E = L->end(); I != E; ++I) {
656     Function *LFn = &*I;
657     LNames.insert(LFn->getName());
658 
659     if (Function *RFn = R->getFunction(LFn->getName()))
660       Queue.push_back(std::make_pair(LFn, RFn));
661     else
662       logf("function %l exists only in left module") << LFn;
663   }
664 
665   for (Module::iterator I = R->begin(), E = R->end(); I != E; ++I) {
666     Function *RFn = &*I;
667     if (!LNames.count(RFn->getName()))
668       logf("function %r exists only in right module") << RFn;
669   }
670 
671   for (SmallVectorImpl<std::pair<Function*,Function*> >::iterator
672          I = Queue.begin(), E = Queue.end(); I != E; ++I)
673     diff(I->first, I->second);
674 }
675 
equivalentAsOperands(GlobalValue * L,GlobalValue * R)676 bool DifferenceEngine::equivalentAsOperands(GlobalValue *L, GlobalValue *R) {
677   if (globalValueOracle) return (*globalValueOracle)(L, R);
678   return L->getName() == R->getName();
679 }
680