blob: 5e41798ad8d4379265b2948fae4fb1d79e8aa6ef [file] [log] [blame]
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001//===- MergeFunctions.cpp - Merge identical functions ---------------------===//
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 pass looks for equivalent functions that are mergable and folds them.
11//
Stepan Dyatkovskiy471eab32014-06-22 00:57:09 +000012// Order relation is defined on set of functions. It was made through
13// special function comparison procedure that returns
14// 0 when functions are equal,
15// -1 when Left function is less than right function, and
16// 1 for opposite case. We need total-ordering, so we need to maintain
17// four properties on the functions set:
18// a <= a (reflexivity)
19// if a <= b and b <= a then a = b (antisymmetry)
20// if a <= b and b <= c then a <= c (transitivity).
21// for all a and b: a <= b or b <= a (totality).
Nick Lewyckyd01d42e2008-11-02 05:52:50 +000022//
Stepan Dyatkovskiy471eab32014-06-22 00:57:09 +000023// Comparison iterates through each instruction in each basic block.
24// Functions are kept on binary tree. For each new function F we perform
25// lookup in binary tree.
26// In practice it works the following way:
27// -- We define Function* container class with custom "operator<" (FunctionPtr).
28// -- "FunctionPtr" instances are stored in std::set collection, so every
29// std::set::insert operation will give you result in log(N) time.
Nick Lewyckyd01d42e2008-11-02 05:52:50 +000030//
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +000031// When a match is found the functions are folded. If both functions are
32// overridable, we move the functionality into a new internal function and
33// leave two overridable thunks to it.
Nick Lewyckyd01d42e2008-11-02 05:52:50 +000034//
35//===----------------------------------------------------------------------===//
36//
37// Future work:
38//
Nick Lewyckyd01d42e2008-11-02 05:52:50 +000039// * virtual functions.
40//
41// Many functions have their address taken by the virtual function table for
42// the object they belong to. However, as long as it's only used for a lookup
Nick Lewyckyfbd27572010-08-08 05:04:23 +000043// and call, this is irrelevant, and we'd like to fold such functions.
Nick Lewyckyd01d42e2008-11-02 05:52:50 +000044//
Nick Lewyckyfbd27572010-08-08 05:04:23 +000045// * be smarter about bitcasts.
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +000046//
47// In order to fold functions, we will sometimes add either bitcast instructions
48// or bitcast constant expressions. Unfortunately, this can confound further
49// analysis since the two functions differ where one has a bitcast and the
Nick Lewyckyfbd27572010-08-08 05:04:23 +000050// other doesn't. We should learn to look through bitcasts.
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +000051//
Stepan Dyatkovskiy471eab32014-06-22 00:57:09 +000052// * Compare complex types with pointer types inside.
53// * Compare cross-reference cases.
54// * Compare complex expressions.
55//
56// All the three issues above could be described as ability to prove that
57// fA == fB == fC == fE == fF == fG in example below:
58//
59// void fA() {
60// fB();
61// }
62// void fB() {
63// fA();
64// }
65//
66// void fE() {
67// fF();
68// }
69// void fF() {
70// fG();
71// }
72// void fG() {
73// fE();
74// }
75//
76// Simplest cross-reference case (fA <--> fB) was implemented in previous
77// versions of MergeFunctions, though it presented only in two function pairs
78// in test-suite (that counts >50k functions)
79// Though possibility to detect complex cross-referencing (e.g.: A->B->C->D->A)
80// could cover much more cases.
81//
Nick Lewyckyd01d42e2008-11-02 05:52:50 +000082//===----------------------------------------------------------------------===//
83
Nick Lewyckyd01d42e2008-11-02 05:52:50 +000084#include "llvm/Transforms/IPO.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000085#include "llvm/ADT/DenseSet.h"
86#include "llvm/ADT/FoldingSet.h"
87#include "llvm/ADT/STLExtras.h"
88#include "llvm/ADT/SmallSet.h"
89#include "llvm/ADT/Statistic.h"
Chandler Carruth219b89b2014-03-04 11:01:28 +000090#include "llvm/IR/CallSite.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000091#include "llvm/IR/Constants.h"
92#include "llvm/IR/DataLayout.h"
93#include "llvm/IR/IRBuilder.h"
94#include "llvm/IR/InlineAsm.h"
95#include "llvm/IR/Instructions.h"
96#include "llvm/IR/LLVMContext.h"
97#include "llvm/IR/Module.h"
98#include "llvm/IR/Operator.h"
Chandler Carruth4220e9c2014-03-04 11:17:44 +000099#include "llvm/IR/ValueHandle.h"
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000100#include "llvm/Pass.h"
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +0000101#include "llvm/Support/CommandLine.h"
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000102#include "llvm/Support/Debug.h"
Torok Edwin56d06592009-07-11 20:10:48 +0000103#include "llvm/Support/ErrorHandling.h"
Daniel Dunbar0dd5e1e2009-07-25 00:23:56 +0000104#include "llvm/Support/raw_ostream.h"
Nick Lewycky68984ed2010-08-31 08:29:37 +0000105#include <vector>
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000106using namespace llvm;
107
Chandler Carruth964daaa2014-04-22 02:55:47 +0000108#define DEBUG_TYPE "mergefunc"
109
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000110STATISTIC(NumFunctionsMerged, "Number of functions merged");
Nick Lewycky71972d42010-09-07 01:42:10 +0000111STATISTIC(NumThunksWritten, "Number of thunks generated");
Nick Lewyckyf1cec162011-01-25 08:56:50 +0000112STATISTIC(NumAliasesWritten, "Number of aliases generated");
Nick Lewycky71972d42010-09-07 01:42:10 +0000113STATISTIC(NumDoubleWeak, "Number of new functions created");
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000114
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +0000115static cl::opt<unsigned> NumFunctionsForSanityCheck(
116 "mergefunc-sanity",
117 cl::desc("How many functions in module could be used for "
118 "MergeFunctions pass sanity check. "
119 "'0' disables this check. Works only with '-debug' key."),
120 cl::init(0), cl::Hidden);
121
Nick Lewyckyf3a07ec2010-09-05 09:00:32 +0000122namespace {
Nick Lewycky00959372010-09-05 08:22:49 +0000123
Nick Lewyckyfbd27572010-08-08 05:04:23 +0000124/// FunctionComparator - Compares two functions to determine whether or not
Micah Villmowcdfe20b2012-10-08 16:38:25 +0000125/// they will generate machine code with the same behaviour. DataLayout is
Nick Lewyckyfbd27572010-08-08 05:04:23 +0000126/// used if available. The comparator always fails conservatively (erring on the
127/// side of claiming that two functions are different).
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +0000128class FunctionComparator {
129public:
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000130 FunctionComparator(const Function *F1, const Function *F2)
131 : FnL(F1), FnR(F2) {}
Nick Lewyckye04dc222009-06-12 08:04:51 +0000132
Nick Lewyckycfb284c2011-01-28 08:43:14 +0000133 /// Test whether the two functions have equivalent behaviour.
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000134 int compare();
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +0000135
136private:
Nick Lewyckycfb284c2011-01-28 08:43:14 +0000137 /// Test whether two basic blocks have equivalent behaviour.
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000138 int compare(const BasicBlock *BBL, const BasicBlock *BBR);
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +0000139
Stepan Dyatkovskiyd1031302014-05-07 09:05:10 +0000140 /// Constants comparison.
141 /// Its analog to lexicographical comparison between hypothetical numbers
142 /// of next format:
143 /// <bitcastability-trait><raw-bit-contents>
144 ///
145 /// 1. Bitcastability.
146 /// Check whether L's type could be losslessly bitcasted to R's type.
147 /// On this stage method, in case when lossless bitcast is not possible
148 /// method returns -1 or 1, thus also defining which type is greater in
149 /// context of bitcastability.
150 /// Stage 0: If types are equal in terms of cmpTypes, then we can go straight
151 /// to the contents comparison.
152 /// If types differ, remember types comparison result and check
153 /// whether we still can bitcast types.
154 /// Stage 1: Types that satisfies isFirstClassType conditions are always
155 /// greater then others.
156 /// Stage 2: Vector is greater then non-vector.
157 /// If both types are vectors, then vector with greater bitwidth is
158 /// greater.
159 /// If both types are vectors with the same bitwidth, then types
160 /// are bitcastable, and we can skip other stages, and go to contents
161 /// comparison.
162 /// Stage 3: Pointer types are greater than non-pointers. If both types are
163 /// pointers of the same address space - go to contents comparison.
164 /// Different address spaces: pointer with greater address space is
165 /// greater.
166 /// Stage 4: Types are neither vectors, nor pointers. And they differ.
167 /// We don't know how to bitcast them. So, we better don't do it,
168 /// and return types comparison result (so it determines the
169 /// relationship among constants we don't know how to bitcast).
170 ///
171 /// Just for clearance, let's see how the set of constants could look
172 /// on single dimension axis:
173 ///
174 /// [NFCT], [FCT, "others"], [FCT, pointers], [FCT, vectors]
175 /// Where: NFCT - Not a FirstClassType
176 /// FCT - FirstClassTyp:
177 ///
178 /// 2. Compare raw contents.
179 /// It ignores types on this stage and only compares bits from L and R.
180 /// Returns 0, if L and R has equivalent contents.
181 /// -1 or 1 if values are different.
182 /// Pretty trivial:
183 /// 2.1. If contents are numbers, compare numbers.
184 /// Ints with greater bitwidth are greater. Ints with same bitwidths
185 /// compared by their contents.
186 /// 2.2. "And so on". Just to avoid discrepancies with comments
187 /// perhaps it would be better to read the implementation itself.
188 /// 3. And again about overall picture. Let's look back at how the ordered set
189 /// of constants will look like:
190 /// [NFCT], [FCT, "others"], [FCT, pointers], [FCT, vectors]
191 ///
192 /// Now look, what could be inside [FCT, "others"], for example:
193 /// [FCT, "others"] =
194 /// [
195 /// [double 0.1], [double 1.23],
196 /// [i32 1], [i32 2],
197 /// { double 1.0 }, ; StructTyID, NumElements = 1
198 /// { i32 1 }, ; StructTyID, NumElements = 1
199 /// { double 1, i32 1 }, ; StructTyID, NumElements = 2
200 /// { i32 1, double 1 } ; StructTyID, NumElements = 2
201 /// ]
202 ///
203 /// Let's explain the order. Float numbers will be less than integers, just
204 /// because of cmpType terms: FloatTyID < IntegerTyID.
205 /// Floats (with same fltSemantics) are sorted according to their value.
206 /// Then you can see integers, and they are, like a floats,
207 /// could be easy sorted among each others.
208 /// The structures. Structures are grouped at the tail, again because of their
209 /// TypeID: StructTyID > IntegerTyID > FloatTyID.
210 /// Structures with greater number of elements are greater. Structures with
211 /// greater elements going first are greater.
212 /// The same logic with vectors, arrays and other possible complex types.
213 ///
214 /// Bitcastable constants.
215 /// Let's assume, that some constant, belongs to some group of
216 /// "so-called-equal" values with different types, and at the same time
217 /// belongs to another group of constants with equal types
218 /// and "really" equal values.
219 ///
220 /// Now, prove that this is impossible:
221 ///
222 /// If constant A with type TyA is bitcastable to B with type TyB, then:
223 /// 1. All constants with equal types to TyA, are bitcastable to B. Since
224 /// those should be vectors (if TyA is vector), pointers
225 /// (if TyA is pointer), or else (if TyA equal to TyB), those types should
226 /// be equal to TyB.
227 /// 2. All constants with non-equal, but bitcastable types to TyA, are
228 /// bitcastable to B.
229 /// Once again, just because we allow it to vectors and pointers only.
230 /// This statement could be expanded as below:
231 /// 2.1. All vectors with equal bitwidth to vector A, has equal bitwidth to
232 /// vector B, and thus bitcastable to B as well.
233 /// 2.2. All pointers of the same address space, no matter what they point to,
234 /// bitcastable. So if C is pointer, it could be bitcasted to A and to B.
235 /// So any constant equal or bitcastable to A is equal or bitcastable to B.
236 /// QED.
237 ///
238 /// In another words, for pointers and vectors, we ignore top-level type and
239 /// look at their particular properties (bit-width for vectors, and
240 /// address space for pointers).
241 /// If these properties are equal - compare their contents.
242 int cmpConstants(const Constant *L, const Constant *R);
243
Nick Lewyckycfb284c2011-01-28 08:43:14 +0000244 /// Assign or look up previously assigned numbers for the two values, and
245 /// return whether the numbers are equal. Numbers are assigned in the order
246 /// visited.
Stepan Dyatkovskiycfd641f2014-05-07 11:11:39 +0000247 /// Comparison order:
248 /// Stage 0: Value that is function itself is always greater then others.
249 /// If left and right values are references to their functions, then
250 /// they are equal.
251 /// Stage 1: Constants are greater than non-constants.
252 /// If both left and right are constants, then the result of
253 /// cmpConstants is used as cmpValues result.
254 /// Stage 2: InlineAsm instances are greater than others. If both left and
255 /// right are InlineAsm instances, InlineAsm* pointers casted to
256 /// integers and compared as numbers.
257 /// Stage 3: For all other cases we compare order we meet these values in
258 /// their functions. If right value was met first during scanning,
259 /// then left value is greater.
260 /// In another words, we compare serial numbers, for more details
261 /// see comments for sn_mapL and sn_mapR.
262 int cmpValues(const Value *L, const Value *R);
263
Nick Lewyckycfb284c2011-01-28 08:43:14 +0000264 /// Compare two Instructions for equivalence, similar to
265 /// Instruction::isSameOperationAs but with modifications to the type
Nick Lewyckyfbd27572010-08-08 05:04:23 +0000266 /// comparison.
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000267 /// Stages are listed in "most significant stage first" order:
268 /// On each stage below, we do comparison between some left and right
269 /// operation parts. If parts are non-equal, we assign parts comparison
270 /// result to the operation comparison result and exit from method.
271 /// Otherwise we proceed to the next stage.
272 /// Stages:
273 /// 1. Operations opcodes. Compared as numbers.
274 /// 2. Number of operands.
275 /// 3. Operation types. Compared with cmpType method.
276 /// 4. Compare operation subclass optional data as stream of bytes:
277 /// just convert it to integers and call cmpNumbers.
278 /// 5. Compare in operation operand types with cmpType in
279 /// most significant operand first order.
280 /// 6. Last stage. Check operations for some specific attributes.
281 /// For example, for Load it would be:
282 /// 6.1.Load: volatile (as boolean flag)
283 /// 6.2.Load: alignment (as integer numbers)
284 /// 6.3.Load: synch-scope (as integer numbers)
Stepan Dyatkovskiy6baeb882014-06-20 19:11:56 +0000285 /// 6.4.Load: range metadata (as integer numbers)
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000286 /// On this stage its better to see the code, since its not more than 10-15
287 /// strings for particular instruction, and could change sometimes.
Stepan Dyatkovskiy87c046182014-07-31 07:16:59 +0000288 int cmpOperations(const Instruction *L, const Instruction *R) const;
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000289
Nick Lewyckycfb284c2011-01-28 08:43:14 +0000290 /// Compare two GEPs for equivalent pointer arithmetic.
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000291 /// Parts to be compared for each comparison stage,
292 /// most significant stage first:
293 /// 1. Address space. As numbers.
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000294 /// 2. Constant offset, (using GEPOperator::accumulateConstantOffset method).
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000295 /// 3. Pointer operand type (using cmpType method).
296 /// 4. Number of operands.
297 /// 5. Compare operands, using cmpValues method.
Stepan Dyatkovskiy016dadd2014-08-25 08:12:45 +0000298 int cmpGEPs(const GEPOperator *GEPL, const GEPOperator *GEPR);
299 int cmpGEPs(const GetElementPtrInst *GEPL, const GetElementPtrInst *GEPR) {
300 return cmpGEPs(cast<GEPOperator>(GEPL), cast<GEPOperator>(GEPR));
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000301 }
302
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000303 /// cmpType - compares two types,
304 /// defines total ordering among the types set.
305 ///
306 /// Return values:
307 /// 0 if types are equal,
308 /// -1 if Left is less than Right,
309 /// +1 if Left is greater than Right.
310 ///
311 /// Description:
312 /// Comparison is broken onto stages. Like in lexicographical comparison
313 /// stage coming first has higher priority.
314 /// On each explanation stage keep in mind total ordering properties.
315 ///
Stepan Dyatkovskiy90c44362014-03-14 08:17:19 +0000316 /// 0. Before comparison we coerce pointer types of 0 address space to
317 /// integer.
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000318 /// We also don't bother with same type at left and right, so
319 /// just return 0 in this case.
320 ///
321 /// 1. If types are of different kind (different type IDs).
322 /// Return result of type IDs comparison, treating them as numbers.
323 /// 2. If types are vectors or integers, compare Type* values as numbers.
324 /// 3. Types has same ID, so check whether they belongs to the next group:
325 /// * Void
326 /// * Float
327 /// * Double
328 /// * X86_FP80
329 /// * FP128
330 /// * PPC_FP128
331 /// * Label
332 /// * Metadata
333 /// If so - return 0, yes - we can treat these types as equal only because
334 /// their IDs are same.
335 /// 4. If Left and Right are pointers, return result of address space
336 /// comparison (numbers comparison). We can treat pointer types of same
337 /// address space as equal.
338 /// 5. If types are complex.
339 /// Then both Left and Right are to be expanded and their element types will
340 /// be checked with the same way. If we get Res != 0 on some stage, return it.
341 /// Otherwise return 0.
342 /// 6. For all other cases put llvm_unreachable.
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000343 int cmpTypes(Type *TyL, Type *TyR) const;
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000344
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000345 int cmpNumbers(uint64_t L, uint64_t R) const;
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +0000346
Stepan Dyatkovskiy7f895c12014-08-25 08:19:50 +0000347 int cmpAPInts(const APInt &L, const APInt &R) const;
Stepan Dyatkovskiyc90308b2014-08-25 08:22:46 +0000348 int cmpAPFloats(const APFloat &L, const APFloat &R) const;
Stepan Dyatkovskiy5c2cc252014-05-16 08:55:34 +0000349 int cmpStrings(StringRef L, StringRef R) const;
350 int cmpAttrs(const AttributeSet L, const AttributeSet R) const;
Stepan Dyatkovskiyd1031302014-05-07 09:05:10 +0000351
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +0000352 // The two functions undergoing comparison.
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000353 const Function *FnL, *FnR;
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +0000354
Stepan Dyatkovskiycfd641f2014-05-07 11:11:39 +0000355 /// Assign serial numbers to values from left function, and values from
356 /// right function.
357 /// Explanation:
358 /// Being comparing functions we need to compare values we meet at left and
359 /// right sides.
360 /// Its easy to sort things out for external values. It just should be
361 /// the same value at left and right.
362 /// But for local values (those were introduced inside function body)
363 /// we have to ensure they were introduced at exactly the same place,
364 /// and plays the same role.
365 /// Let's assign serial number to each value when we meet it first time.
366 /// Values that were met at same place will be with same serial numbers.
367 /// In this case it would be good to explain few points about values assigned
368 /// to BBs and other ways of implementation (see below).
369 ///
370 /// 1. Safety of BB reordering.
371 /// It's safe to change the order of BasicBlocks in function.
372 /// Relationship with other functions and serial numbering will not be
373 /// changed in this case.
374 /// As follows from FunctionComparator::compare(), we do CFG walk: we start
375 /// from the entry, and then take each terminator. So it doesn't matter how in
376 /// fact BBs are ordered in function. And since cmpValues are called during
377 /// this walk, the numbering depends only on how BBs located inside the CFG.
378 /// So the answer is - yes. We will get the same numbering.
379 ///
380 /// 2. Impossibility to use dominance properties of values.
381 /// If we compare two instruction operands: first is usage of local
382 /// variable AL from function FL, and second is usage of local variable AR
383 /// from FR, we could compare their origins and check whether they are
384 /// defined at the same place.
385 /// But, we are still not able to compare operands of PHI nodes, since those
386 /// could be operands from further BBs we didn't scan yet.
387 /// So it's impossible to use dominance properties in general.
388 DenseMap<const Value*, int> sn_mapL, sn_mapR;
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +0000389};
Nick Lewycky564fcca2011-01-28 07:36:21 +0000390
Stepan Dyatkovskiyfe134cd2014-09-10 10:08:25 +0000391class FunctionNode {
Arnold Schwaighofer0302da62015-06-09 00:03:29 +0000392 mutable AssertingVH<Function> F;
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +0000393
394public:
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000395 FunctionNode(Function *F) : F(F) {}
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +0000396 Function *getFunc() const { return F; }
Arnold Schwaighofer0302da62015-06-09 00:03:29 +0000397
398 /// Replace the reference to the function F by the function G, assuming their
399 /// implementations are equal.
400 void replaceBy(Function *G) const {
401 assert(!(*this < FunctionNode(G)) && !(FunctionNode(G) < *this) &&
402 "The two functions must be equal");
403
404 F = G;
405 }
406
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +0000407 void release() { F = 0; }
Stepan Dyatkovskiyfe134cd2014-09-10 10:08:25 +0000408 bool operator<(const FunctionNode &RHS) const {
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000409 return (FunctionComparator(F, RHS.getFunc()).compare()) == -1;
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +0000410 }
411};
Alexander Kornienko70bc5f12015-06-19 15:57:42 +0000412} // namespace
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +0000413
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000414int FunctionComparator::cmpNumbers(uint64_t L, uint64_t R) const {
415 if (L < R) return -1;
416 if (L > R) return 1;
417 return 0;
418}
Stepan Dyatkovskiyabb85052013-11-26 16:11:03 +0000419
Stepan Dyatkovskiy7f895c12014-08-25 08:19:50 +0000420int FunctionComparator::cmpAPInts(const APInt &L, const APInt &R) const {
Stepan Dyatkovskiyd1031302014-05-07 09:05:10 +0000421 if (int Res = cmpNumbers(L.getBitWidth(), R.getBitWidth()))
422 return Res;
423 if (L.ugt(R)) return 1;
424 if (R.ugt(L)) return -1;
425 return 0;
426}
427
Stepan Dyatkovskiyc90308b2014-08-25 08:22:46 +0000428int FunctionComparator::cmpAPFloats(const APFloat &L, const APFloat &R) const {
Stepan Dyatkovskiyd1031302014-05-07 09:05:10 +0000429 if (int Res = cmpNumbers((uint64_t)&L.getSemantics(),
430 (uint64_t)&R.getSemantics()))
431 return Res;
Stepan Dyatkovskiy7f895c12014-08-25 08:19:50 +0000432 return cmpAPInts(L.bitcastToAPInt(), R.bitcastToAPInt());
Stepan Dyatkovskiyd1031302014-05-07 09:05:10 +0000433}
434
Stepan Dyatkovskiy5c2cc252014-05-16 08:55:34 +0000435int FunctionComparator::cmpStrings(StringRef L, StringRef R) const {
436 // Prevent heavy comparison, compare sizes first.
437 if (int Res = cmpNumbers(L.size(), R.size()))
438 return Res;
439
440 // Compare strings lexicographically only when it is necessary: only when
441 // strings are equal in size.
442 return L.compare(R);
443}
444
445int FunctionComparator::cmpAttrs(const AttributeSet L,
446 const AttributeSet R) const {
447 if (int Res = cmpNumbers(L.getNumSlots(), R.getNumSlots()))
448 return Res;
449
450 for (unsigned i = 0, e = L.getNumSlots(); i != e; ++i) {
451 AttributeSet::iterator LI = L.begin(i), LE = L.end(i), RI = R.begin(i),
452 RE = R.end(i);
453 for (; LI != LE && RI != RE; ++LI, ++RI) {
454 Attribute LA = *LI;
455 Attribute RA = *RI;
456 if (LA < RA)
457 return -1;
458 if (RA < LA)
459 return 1;
460 }
461 if (LI != LE)
462 return 1;
463 if (RI != RE)
464 return -1;
465 }
466 return 0;
467}
468
Stepan Dyatkovskiyd1031302014-05-07 09:05:10 +0000469/// Constants comparison:
470/// 1. Check whether type of L constant could be losslessly bitcasted to R
471/// type.
472/// 2. Compare constant contents.
473/// For more details see declaration comments.
474int FunctionComparator::cmpConstants(const Constant *L, const Constant *R) {
475
476 Type *TyL = L->getType();
477 Type *TyR = R->getType();
478
479 // Check whether types are bitcastable. This part is just re-factored
480 // Type::canLosslesslyBitCastTo method, but instead of returning true/false,
481 // we also pack into result which type is "less" for us.
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000482 int TypesRes = cmpTypes(TyL, TyR);
Stepan Dyatkovskiyd1031302014-05-07 09:05:10 +0000483 if (TypesRes != 0) {
484 // Types are different, but check whether we can bitcast them.
485 if (!TyL->isFirstClassType()) {
486 if (TyR->isFirstClassType())
487 return -1;
488 // Neither TyL nor TyR are values of first class type. Return the result
489 // of comparing the types
490 return TypesRes;
491 }
492 if (!TyR->isFirstClassType()) {
493 if (TyL->isFirstClassType())
494 return 1;
495 return TypesRes;
496 }
497
498 // Vector -> Vector conversions are always lossless if the two vector types
499 // have the same size, otherwise not.
500 unsigned TyLWidth = 0;
501 unsigned TyRWidth = 0;
502
503 if (const VectorType *VecTyL = dyn_cast<VectorType>(TyL))
504 TyLWidth = VecTyL->getBitWidth();
505 if (const VectorType *VecTyR = dyn_cast<VectorType>(TyR))
506 TyRWidth = VecTyR->getBitWidth();
507
508 if (TyLWidth != TyRWidth)
509 return cmpNumbers(TyLWidth, TyRWidth);
510
511 // Zero bit-width means neither TyL nor TyR are vectors.
512 if (!TyLWidth) {
513 PointerType *PTyL = dyn_cast<PointerType>(TyL);
514 PointerType *PTyR = dyn_cast<PointerType>(TyR);
515 if (PTyL && PTyR) {
516 unsigned AddrSpaceL = PTyL->getAddressSpace();
517 unsigned AddrSpaceR = PTyR->getAddressSpace();
518 if (int Res = cmpNumbers(AddrSpaceL, AddrSpaceR))
519 return Res;
520 }
521 if (PTyL)
522 return 1;
523 if (PTyR)
524 return -1;
525
526 // TyL and TyR aren't vectors, nor pointers. We don't know how to
527 // bitcast them.
528 return TypesRes;
529 }
530 }
531
532 // OK, types are bitcastable, now check constant contents.
533
534 if (L->isNullValue() && R->isNullValue())
535 return TypesRes;
536 if (L->isNullValue() && !R->isNullValue())
537 return 1;
538 if (!L->isNullValue() && R->isNullValue())
539 return -1;
540
541 if (int Res = cmpNumbers(L->getValueID(), R->getValueID()))
542 return Res;
543
544 switch (L->getValueID()) {
545 case Value::UndefValueVal: return TypesRes;
546 case Value::ConstantIntVal: {
547 const APInt &LInt = cast<ConstantInt>(L)->getValue();
548 const APInt &RInt = cast<ConstantInt>(R)->getValue();
Stepan Dyatkovskiy7f895c12014-08-25 08:19:50 +0000549 return cmpAPInts(LInt, RInt);
Stepan Dyatkovskiyd1031302014-05-07 09:05:10 +0000550 }
551 case Value::ConstantFPVal: {
552 const APFloat &LAPF = cast<ConstantFP>(L)->getValueAPF();
553 const APFloat &RAPF = cast<ConstantFP>(R)->getValueAPF();
Stepan Dyatkovskiyc90308b2014-08-25 08:22:46 +0000554 return cmpAPFloats(LAPF, RAPF);
Stepan Dyatkovskiyd1031302014-05-07 09:05:10 +0000555 }
556 case Value::ConstantArrayVal: {
557 const ConstantArray *LA = cast<ConstantArray>(L);
558 const ConstantArray *RA = cast<ConstantArray>(R);
559 uint64_t NumElementsL = cast<ArrayType>(TyL)->getNumElements();
560 uint64_t NumElementsR = cast<ArrayType>(TyR)->getNumElements();
561 if (int Res = cmpNumbers(NumElementsL, NumElementsR))
562 return Res;
563 for (uint64_t i = 0; i < NumElementsL; ++i) {
564 if (int Res = cmpConstants(cast<Constant>(LA->getOperand(i)),
565 cast<Constant>(RA->getOperand(i))))
566 return Res;
567 }
568 return 0;
569 }
570 case Value::ConstantStructVal: {
571 const ConstantStruct *LS = cast<ConstantStruct>(L);
572 const ConstantStruct *RS = cast<ConstantStruct>(R);
573 unsigned NumElementsL = cast<StructType>(TyL)->getNumElements();
574 unsigned NumElementsR = cast<StructType>(TyR)->getNumElements();
575 if (int Res = cmpNumbers(NumElementsL, NumElementsR))
576 return Res;
577 for (unsigned i = 0; i != NumElementsL; ++i) {
578 if (int Res = cmpConstants(cast<Constant>(LS->getOperand(i)),
579 cast<Constant>(RS->getOperand(i))))
580 return Res;
581 }
582 return 0;
583 }
584 case Value::ConstantVectorVal: {
585 const ConstantVector *LV = cast<ConstantVector>(L);
586 const ConstantVector *RV = cast<ConstantVector>(R);
587 unsigned NumElementsL = cast<VectorType>(TyL)->getNumElements();
588 unsigned NumElementsR = cast<VectorType>(TyR)->getNumElements();
589 if (int Res = cmpNumbers(NumElementsL, NumElementsR))
590 return Res;
591 for (uint64_t i = 0; i < NumElementsL; ++i) {
592 if (int Res = cmpConstants(cast<Constant>(LV->getOperand(i)),
593 cast<Constant>(RV->getOperand(i))))
594 return Res;
595 }
596 return 0;
597 }
598 case Value::ConstantExprVal: {
599 const ConstantExpr *LE = cast<ConstantExpr>(L);
600 const ConstantExpr *RE = cast<ConstantExpr>(R);
601 unsigned NumOperandsL = LE->getNumOperands();
602 unsigned NumOperandsR = RE->getNumOperands();
603 if (int Res = cmpNumbers(NumOperandsL, NumOperandsR))
604 return Res;
605 for (unsigned i = 0; i < NumOperandsL; ++i) {
606 if (int Res = cmpConstants(cast<Constant>(LE->getOperand(i)),
607 cast<Constant>(RE->getOperand(i))))
608 return Res;
609 }
610 return 0;
611 }
612 case Value::FunctionVal:
613 case Value::GlobalVariableVal:
614 case Value::GlobalAliasVal:
615 default: // Unknown constant, cast L and R pointers to numbers and compare.
616 return cmpNumbers((uint64_t)L, (uint64_t)R);
617 }
618}
619
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000620/// cmpType - compares two types,
621/// defines total ordering among the types set.
622/// See method declaration comments for more details.
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000623int FunctionComparator::cmpTypes(Type *TyL, Type *TyR) const {
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000624
Stepan Dyatkovskiya53cf972014-03-14 08:48:52 +0000625 PointerType *PTyL = dyn_cast<PointerType>(TyL);
626 PointerType *PTyR = dyn_cast<PointerType>(TyR);
Stepan Dyatkovskiyabb85052013-11-26 16:11:03 +0000627
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000628 const DataLayout &DL = FnL->getParent()->getDataLayout();
629 if (PTyL && PTyL->getAddressSpace() == 0)
630 TyL = DL.getIntPtrType(TyL);
631 if (PTyR && PTyR->getAddressSpace() == 0)
632 TyR = DL.getIntPtrType(TyR);
Stepan Dyatkovskiyabb85052013-11-26 16:11:03 +0000633
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000634 if (TyL == TyR)
635 return 0;
Matt Arsenault5bcefab2013-11-10 01:44:37 +0000636
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000637 if (int Res = cmpNumbers(TyL->getTypeID(), TyR->getTypeID()))
638 return Res;
Nick Lewyckye04dc222009-06-12 08:04:51 +0000639
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000640 switch (TyL->getTypeID()) {
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000641 default:
642 llvm_unreachable("Unknown type!");
Duncan Sands408bb192010-07-07 07:48:00 +0000643 // Fall through in Release mode.
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000644 case Type::IntegerTyID:
Nick Lewyckyfb622f92011-01-26 08:50:18 +0000645 case Type::VectorTyID:
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000646 // TyL == TyR would have returned true earlier.
647 return cmpNumbers((uint64_t)TyL, (uint64_t)TyR);
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000648
Nick Lewyckye04dc222009-06-12 08:04:51 +0000649 case Type::VoidTyID:
650 case Type::FloatTyID:
651 case Type::DoubleTyID:
652 case Type::X86_FP80TyID:
653 case Type::FP128TyID:
654 case Type::PPC_FP128TyID:
655 case Type::LabelTyID:
656 case Type::MetadataTyID:
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000657 return 0;
Nick Lewyckye04dc222009-06-12 08:04:51 +0000658
Nick Lewyckye04dc222009-06-12 08:04:51 +0000659 case Type::PointerTyID: {
Stepan Dyatkovskiya53cf972014-03-14 08:48:52 +0000660 assert(PTyL && PTyR && "Both types must be pointers here.");
661 return cmpNumbers(PTyL->getAddressSpace(), PTyR->getAddressSpace());
Nick Lewyckye04dc222009-06-12 08:04:51 +0000662 }
663
664 case Type::StructTyID: {
Stepan Dyatkovskiya53cf972014-03-14 08:48:52 +0000665 StructType *STyL = cast<StructType>(TyL);
666 StructType *STyR = cast<StructType>(TyR);
667 if (STyL->getNumElements() != STyR->getNumElements())
668 return cmpNumbers(STyL->getNumElements(), STyR->getNumElements());
Nick Lewyckye04dc222009-06-12 08:04:51 +0000669
Stepan Dyatkovskiya53cf972014-03-14 08:48:52 +0000670 if (STyL->isPacked() != STyR->isPacked())
671 return cmpNumbers(STyL->isPacked(), STyR->isPacked());
Nick Lewyckye04dc222009-06-12 08:04:51 +0000672
Stepan Dyatkovskiya53cf972014-03-14 08:48:52 +0000673 for (unsigned i = 0, e = STyL->getNumElements(); i != e; ++i) {
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000674 if (int Res = cmpTypes(STyL->getElementType(i), STyR->getElementType(i)))
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000675 return Res;
Nick Lewyckye04dc222009-06-12 08:04:51 +0000676 }
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000677 return 0;
Nick Lewyckye04dc222009-06-12 08:04:51 +0000678 }
679
680 case Type::FunctionTyID: {
Stepan Dyatkovskiya53cf972014-03-14 08:48:52 +0000681 FunctionType *FTyL = cast<FunctionType>(TyL);
682 FunctionType *FTyR = cast<FunctionType>(TyR);
683 if (FTyL->getNumParams() != FTyR->getNumParams())
684 return cmpNumbers(FTyL->getNumParams(), FTyR->getNumParams());
Nick Lewyckye04dc222009-06-12 08:04:51 +0000685
Stepan Dyatkovskiya53cf972014-03-14 08:48:52 +0000686 if (FTyL->isVarArg() != FTyR->isVarArg())
687 return cmpNumbers(FTyL->isVarArg(), FTyR->isVarArg());
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000688
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000689 if (int Res = cmpTypes(FTyL->getReturnType(), FTyR->getReturnType()))
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000690 return Res;
Nick Lewyckye04dc222009-06-12 08:04:51 +0000691
Stepan Dyatkovskiya53cf972014-03-14 08:48:52 +0000692 for (unsigned i = 0, e = FTyL->getNumParams(); i != e; ++i) {
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000693 if (int Res = cmpTypes(FTyL->getParamType(i), FTyR->getParamType(i)))
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000694 return Res;
Nick Lewyckye04dc222009-06-12 08:04:51 +0000695 }
Stepan Dyatkovskiyd8eb0bc2014-03-13 11:54:50 +0000696 return 0;
Nick Lewyckye04dc222009-06-12 08:04:51 +0000697 }
698
Nick Lewycky375efe32010-07-16 06:31:12 +0000699 case Type::ArrayTyID: {
Stepan Dyatkovskiya53cf972014-03-14 08:48:52 +0000700 ArrayType *ATyL = cast<ArrayType>(TyL);
701 ArrayType *ATyR = cast<ArrayType>(TyR);
702 if (ATyL->getNumElements() != ATyR->getNumElements())
703 return cmpNumbers(ATyL->getNumElements(), ATyR->getNumElements());
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000704 return cmpTypes(ATyL->getElementType(), ATyR->getElementType());
Nick Lewycky375efe32010-07-16 06:31:12 +0000705 }
Nick Lewyckye04dc222009-06-12 08:04:51 +0000706 }
707}
708
Nick Lewyckycfb284c2011-01-28 08:43:14 +0000709// Determine whether the two operations are the same except that pointer-to-A
710// and pointer-to-B are equivalent. This should be kept in sync with
711// Instruction::isSameOperationAs.
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000712// Read method declaration comments for more details.
Stepan Dyatkovskiy87c046182014-07-31 07:16:59 +0000713int FunctionComparator::cmpOperations(const Instruction *L,
714 const Instruction *R) const {
Nick Lewyckycb1a4c22011-02-06 05:04:00 +0000715 // Differences from Instruction::isSameOperationAs:
716 // * replace type comparison with calls to isEquivalentType.
717 // * we test for I->hasSameSubclassOptionalData (nuw/nsw/tail) at the top
718 // * because of the above, we don't test for the tail bit on calls later on
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000719 if (int Res = cmpNumbers(L->getOpcode(), R->getOpcode()))
720 return Res;
721
722 if (int Res = cmpNumbers(L->getNumOperands(), R->getNumOperands()))
723 return Res;
724
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000725 if (int Res = cmpTypes(L->getType(), R->getType()))
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000726 return Res;
727
728 if (int Res = cmpNumbers(L->getRawSubclassOptionalData(),
729 R->getRawSubclassOptionalData()))
730 return Res;
Nick Lewyckye04dc222009-06-12 08:04:51 +0000731
Arnold Schwaighofer6a8c5f62015-05-12 21:42:22 +0000732 if (const AllocaInst *AI = dyn_cast<AllocaInst>(L)) {
733 if (int Res = cmpTypes(AI->getAllocatedType(),
734 cast<AllocaInst>(R)->getAllocatedType()))
735 return Res;
736 if (int Res =
737 cmpNumbers(AI->getAlignment(), cast<AllocaInst>(R)->getAlignment()))
738 return Res;
739 }
740
Nick Lewyckye04dc222009-06-12 08:04:51 +0000741 // We have two instructions of identical opcode and #operands. Check to see
742 // if all operands are the same type
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000743 for (unsigned i = 0, e = L->getNumOperands(); i != e; ++i) {
744 if (int Res =
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000745 cmpTypes(L->getOperand(i)->getType(), R->getOperand(i)->getType()))
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000746 return Res;
747 }
Nick Lewyckye04dc222009-06-12 08:04:51 +0000748
749 // Check special state that is a part of some instructions.
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000750 if (const LoadInst *LI = dyn_cast<LoadInst>(L)) {
751 if (int Res = cmpNumbers(LI->isVolatile(), cast<LoadInst>(R)->isVolatile()))
752 return Res;
753 if (int Res =
754 cmpNumbers(LI->getAlignment(), cast<LoadInst>(R)->getAlignment()))
755 return Res;
756 if (int Res =
757 cmpNumbers(LI->getOrdering(), cast<LoadInst>(R)->getOrdering()))
758 return Res;
Stepan Dyatkovskiy6baeb882014-06-20 19:11:56 +0000759 if (int Res =
760 cmpNumbers(LI->getSynchScope(), cast<LoadInst>(R)->getSynchScope()))
761 return Res;
762 return cmpNumbers((uint64_t)LI->getMetadata(LLVMContext::MD_range),
763 (uint64_t)cast<LoadInst>(R)->getMetadata(LLVMContext::MD_range));
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000764 }
765 if (const StoreInst *SI = dyn_cast<StoreInst>(L)) {
766 if (int Res =
767 cmpNumbers(SI->isVolatile(), cast<StoreInst>(R)->isVolatile()))
768 return Res;
769 if (int Res =
770 cmpNumbers(SI->getAlignment(), cast<StoreInst>(R)->getAlignment()))
771 return Res;
772 if (int Res =
773 cmpNumbers(SI->getOrdering(), cast<StoreInst>(R)->getOrdering()))
774 return Res;
775 return cmpNumbers(SI->getSynchScope(), cast<StoreInst>(R)->getSynchScope());
776 }
777 if (const CmpInst *CI = dyn_cast<CmpInst>(L))
778 return cmpNumbers(CI->getPredicate(), cast<CmpInst>(R)->getPredicate());
779 if (const CallInst *CI = dyn_cast<CallInst>(L)) {
780 if (int Res = cmpNumbers(CI->getCallingConv(),
781 cast<CallInst>(R)->getCallingConv()))
782 return Res;
Stepan Dyatkovskiydee612d2014-07-15 10:46:51 +0000783 if (int Res =
784 cmpAttrs(CI->getAttributes(), cast<CallInst>(R)->getAttributes()))
785 return Res;
786 return cmpNumbers(
787 (uint64_t)CI->getMetadata(LLVMContext::MD_range),
788 (uint64_t)cast<CallInst>(R)->getMetadata(LLVMContext::MD_range));
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000789 }
790 if (const InvokeInst *CI = dyn_cast<InvokeInst>(L)) {
791 if (int Res = cmpNumbers(CI->getCallingConv(),
792 cast<InvokeInst>(R)->getCallingConv()))
793 return Res;
Stepan Dyatkovskiydee612d2014-07-15 10:46:51 +0000794 if (int Res =
795 cmpAttrs(CI->getAttributes(), cast<InvokeInst>(R)->getAttributes()))
796 return Res;
797 return cmpNumbers(
798 (uint64_t)CI->getMetadata(LLVMContext::MD_range),
799 (uint64_t)cast<InvokeInst>(R)->getMetadata(LLVMContext::MD_range));
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000800 }
801 if (const InsertValueInst *IVI = dyn_cast<InsertValueInst>(L)) {
802 ArrayRef<unsigned> LIndices = IVI->getIndices();
803 ArrayRef<unsigned> RIndices = cast<InsertValueInst>(R)->getIndices();
804 if (int Res = cmpNumbers(LIndices.size(), RIndices.size()))
805 return Res;
806 for (size_t i = 0, e = LIndices.size(); i != e; ++i) {
807 if (int Res = cmpNumbers(LIndices[i], RIndices[i]))
808 return Res;
809 }
810 }
811 if (const ExtractValueInst *EVI = dyn_cast<ExtractValueInst>(L)) {
812 ArrayRef<unsigned> LIndices = EVI->getIndices();
813 ArrayRef<unsigned> RIndices = cast<ExtractValueInst>(R)->getIndices();
814 if (int Res = cmpNumbers(LIndices.size(), RIndices.size()))
815 return Res;
816 for (size_t i = 0, e = LIndices.size(); i != e; ++i) {
817 if (int Res = cmpNumbers(LIndices[i], RIndices[i]))
818 return Res;
819 }
820 }
821 if (const FenceInst *FI = dyn_cast<FenceInst>(L)) {
822 if (int Res =
823 cmpNumbers(FI->getOrdering(), cast<FenceInst>(R)->getOrdering()))
824 return Res;
825 return cmpNumbers(FI->getSynchScope(), cast<FenceInst>(R)->getSynchScope());
826 }
Nick Lewyckye04dc222009-06-12 08:04:51 +0000827
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000828 if (const AtomicCmpXchgInst *CXI = dyn_cast<AtomicCmpXchgInst>(L)) {
829 if (int Res = cmpNumbers(CXI->isVolatile(),
830 cast<AtomicCmpXchgInst>(R)->isVolatile()))
831 return Res;
Tim Northover420a2162014-06-13 14:24:07 +0000832 if (int Res = cmpNumbers(CXI->isWeak(),
833 cast<AtomicCmpXchgInst>(R)->isWeak()))
834 return Res;
Stepan Dyatkovskiyfa6820a2014-05-16 11:02:22 +0000835 if (int Res = cmpNumbers(CXI->getSuccessOrdering(),
836 cast<AtomicCmpXchgInst>(R)->getSuccessOrdering()))
837 return Res;
838 if (int Res = cmpNumbers(CXI->getFailureOrdering(),
839 cast<AtomicCmpXchgInst>(R)->getFailureOrdering()))
840 return Res;
841 return cmpNumbers(CXI->getSynchScope(),
842 cast<AtomicCmpXchgInst>(R)->getSynchScope());
843 }
844 if (const AtomicRMWInst *RMWI = dyn_cast<AtomicRMWInst>(L)) {
845 if (int Res = cmpNumbers(RMWI->getOperation(),
846 cast<AtomicRMWInst>(R)->getOperation()))
847 return Res;
848 if (int Res = cmpNumbers(RMWI->isVolatile(),
849 cast<AtomicRMWInst>(R)->isVolatile()))
850 return Res;
851 if (int Res = cmpNumbers(RMWI->getOrdering(),
852 cast<AtomicRMWInst>(R)->getOrdering()))
853 return Res;
854 return cmpNumbers(RMWI->getSynchScope(),
855 cast<AtomicRMWInst>(R)->getSynchScope());
856 }
857 return 0;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000858}
859
Nick Lewyckycfb284c2011-01-28 08:43:14 +0000860// Determine whether two GEP operations perform the same underlying arithmetic.
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000861// Read method declaration comments for more details.
Stepan Dyatkovskiy016dadd2014-08-25 08:12:45 +0000862int FunctionComparator::cmpGEPs(const GEPOperator *GEPL,
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000863 const GEPOperator *GEPR) {
Matt Arsenault5bcefab2013-11-10 01:44:37 +0000864
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000865 unsigned int ASL = GEPL->getPointerAddressSpace();
866 unsigned int ASR = GEPR->getPointerAddressSpace();
867
868 if (int Res = cmpNumbers(ASL, ASR))
869 return Res;
870
871 // When we have target data, we can reduce the GEP down to the value in bytes
872 // added to the address.
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000873 const DataLayout &DL = FnL->getParent()->getDataLayout();
874 unsigned BitWidth = DL.getPointerSizeInBits(ASL);
875 APInt OffsetL(BitWidth, 0), OffsetR(BitWidth, 0);
876 if (GEPL->accumulateConstantOffset(DL, OffsetL) &&
877 GEPR->accumulateConstantOffset(DL, OffsetR))
878 return cmpAPInts(OffsetL, OffsetR);
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000879
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000880 if (int Res = cmpNumbers((uint64_t)GEPL->getPointerOperand()->getType(),
881 (uint64_t)GEPR->getPointerOperand()->getType()))
882 return Res;
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000883
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000884 if (int Res = cmpNumbers(GEPL->getNumOperands(), GEPR->getNumOperands()))
885 return Res;
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000886
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000887 for (unsigned i = 0, e = GEPL->getNumOperands(); i != e; ++i) {
888 if (int Res = cmpValues(GEPL->getOperand(i), GEPR->getOperand(i)))
889 return Res;
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000890 }
891
Stepan Dyatkovskiy948366a2014-05-16 11:55:02 +0000892 return 0;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000893}
894
Stepan Dyatkovskiycfd641f2014-05-07 11:11:39 +0000895/// Compare two values used by the two functions under pair-wise comparison. If
896/// this is the first time the values are seen, they're added to the mapping so
897/// that we will detect mismatches on next use.
898/// See comments in declaration for more details.
899int FunctionComparator::cmpValues(const Value *L, const Value *R) {
900 // Catch self-reference case.
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000901 if (L == FnL) {
902 if (R == FnR)
Stepan Dyatkovskiycfd641f2014-05-07 11:11:39 +0000903 return 0;
904 return -1;
905 }
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000906 if (R == FnR) {
907 if (L == FnL)
Stepan Dyatkovskiycfd641f2014-05-07 11:11:39 +0000908 return 0;
909 return 1;
Nick Lewycky13e04ae2011-01-27 08:38:19 +0000910 }
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000911
Stepan Dyatkovskiycfd641f2014-05-07 11:11:39 +0000912 const Constant *ConstL = dyn_cast<Constant>(L);
913 const Constant *ConstR = dyn_cast<Constant>(R);
914 if (ConstL && ConstR) {
915 if (L == R)
916 return 0;
917 return cmpConstants(ConstL, ConstR);
918 }
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000919
Stepan Dyatkovskiycfd641f2014-05-07 11:11:39 +0000920 if (ConstL)
921 return 1;
922 if (ConstR)
923 return -1;
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000924
Stepan Dyatkovskiycfd641f2014-05-07 11:11:39 +0000925 const InlineAsm *InlineAsmL = dyn_cast<InlineAsm>(L);
926 const InlineAsm *InlineAsmR = dyn_cast<InlineAsm>(R);
927
928 if (InlineAsmL && InlineAsmR)
929 return cmpNumbers((uint64_t)L, (uint64_t)R);
930 if (InlineAsmL)
931 return 1;
932 if (InlineAsmR)
933 return -1;
934
935 auto LeftSN = sn_mapL.insert(std::make_pair(L, sn_mapL.size())),
936 RightSN = sn_mapR.insert(std::make_pair(R, sn_mapR.size()));
937
938 return cmpNumbers(LeftSN.first->second, RightSN.first->second);
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000939}
Nick Lewyckycfb284c2011-01-28 08:43:14 +0000940// Test whether two basic blocks have equivalent behaviour.
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000941int FunctionComparator::compare(const BasicBlock *BBL, const BasicBlock *BBR) {
942 BasicBlock::const_iterator InstL = BBL->begin(), InstLE = BBL->end();
943 BasicBlock::const_iterator InstR = BBR->begin(), InstRE = BBR->end();
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000944
945 do {
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000946 if (int Res = cmpValues(InstL, InstR))
947 return Res;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000948
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000949 const GetElementPtrInst *GEPL = dyn_cast<GetElementPtrInst>(InstL);
950 const GetElementPtrInst *GEPR = dyn_cast<GetElementPtrInst>(InstR);
Nick Lewycky47b71c52009-06-13 19:09:52 +0000951
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000952 if (GEPL && !GEPR)
953 return 1;
954 if (GEPR && !GEPL)
955 return -1;
Nick Lewycky47b71c52009-06-13 19:09:52 +0000956
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000957 if (GEPL && GEPR) {
958 if (int Res =
959 cmpValues(GEPL->getPointerOperand(), GEPR->getPointerOperand()))
960 return Res;
Stepan Dyatkovskiy016dadd2014-08-25 08:12:45 +0000961 if (int Res = cmpGEPs(GEPL, GEPR))
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000962 return Res;
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +0000963 } else {
Stepan Dyatkovskiy87c046182014-07-31 07:16:59 +0000964 if (int Res = cmpOperations(InstL, InstR))
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000965 return Res;
966 assert(InstL->getNumOperands() == InstR->getNumOperands());
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000967
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000968 for (unsigned i = 0, e = InstL->getNumOperands(); i != e; ++i) {
969 Value *OpL = InstL->getOperand(i);
970 Value *OpR = InstR->getOperand(i);
971 if (int Res = cmpValues(OpL, OpR))
972 return Res;
973 if (int Res = cmpNumbers(OpL->getValueID(), OpR->getValueID()))
974 return Res;
975 // TODO: Already checked in cmpOperation
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +0000976 if (int Res = cmpTypes(OpL->getType(), OpR->getType()))
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000977 return Res;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000978 }
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000979 }
980
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000981 ++InstL, ++InstR;
982 } while (InstL != InstLE && InstR != InstRE);
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000983
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000984 if (InstL != InstLE && InstR == InstRE)
985 return 1;
986 if (InstL == InstLE && InstR != InstRE)
987 return -1;
988 return 0;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000989}
990
Nick Lewyckycfb284c2011-01-28 08:43:14 +0000991// Test whether the two functions have equivalent behaviour.
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000992int FunctionComparator::compare() {
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000993
Stepan Dyatkovskiycfd641f2014-05-07 11:11:39 +0000994 sn_mapL.clear();
995 sn_mapR.clear();
996
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +0000997 if (int Res = cmpAttrs(FnL->getAttributes(), FnR->getAttributes()))
998 return Res;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +0000999
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001000 if (int Res = cmpNumbers(FnL->hasGC(), FnR->hasGC()))
1001 return Res;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001002
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001003 if (FnL->hasGC()) {
1004 if (int Res = cmpNumbers((uint64_t)FnL->getGC(), (uint64_t)FnR->getGC()))
1005 return Res;
1006 }
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001007
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001008 if (int Res = cmpNumbers(FnL->hasSection(), FnR->hasSection()))
1009 return Res;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001010
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001011 if (FnL->hasSection()) {
1012 if (int Res = cmpStrings(FnL->getSection(), FnR->getSection()))
1013 return Res;
1014 }
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001015
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001016 if (int Res = cmpNumbers(FnL->isVarArg(), FnR->isVarArg()))
1017 return Res;
Nick Lewyckye04dc222009-06-12 08:04:51 +00001018
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001019 // TODO: if it's internal and only used in direct calls, we could handle this
1020 // case too.
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001021 if (int Res = cmpNumbers(FnL->getCallingConv(), FnR->getCallingConv()))
1022 return Res;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001023
Stepan Dyatkovskiy0b765de2014-08-25 08:16:39 +00001024 if (int Res = cmpTypes(FnL->getFunctionType(), FnR->getFunctionType()))
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001025 return Res;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001026
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001027 assert(FnL->arg_size() == FnR->arg_size() &&
Nick Lewycky71972d42010-09-07 01:42:10 +00001028 "Identically typed functions have different numbers of args!");
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001029
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +00001030 // Visit the arguments so that they get enumerated in the order they're
1031 // passed in.
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001032 for (Function::const_arg_iterator ArgLI = FnL->arg_begin(),
1033 ArgRI = FnR->arg_begin(),
1034 ArgLE = FnL->arg_end();
1035 ArgLI != ArgLE; ++ArgLI, ++ArgRI) {
1036 if (cmpValues(ArgLI, ArgRI) != 0)
Nick Lewycky71972d42010-09-07 01:42:10 +00001037 llvm_unreachable("Arguments repeat!");
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001038 }
1039
Nick Lewyckyfbd27572010-08-08 05:04:23 +00001040 // We do a CFG-ordered walk since the actual ordering of the blocks in the
1041 // linked list is immaterial. Our walk starts at the entry block for both
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +00001042 // functions, then takes each block from each terminator in order. As an
1043 // artifact, this also means that unreachable blocks are ignored.
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001044 SmallVector<const BasicBlock *, 8> FnLBBs, FnRBBs;
Nick Lewyckyf52bd9c2010-08-02 05:23:03 +00001045 SmallSet<const BasicBlock *, 128> VisitedBBs; // in terms of F1.
Nick Lewyckyf216f69a2010-08-06 07:21:30 +00001046
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001047 FnLBBs.push_back(&FnL->getEntryBlock());
1048 FnRBBs.push_back(&FnR->getEntryBlock());
Nick Lewyckyf216f69a2010-08-06 07:21:30 +00001049
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001050 VisitedBBs.insert(FnLBBs[0]);
1051 while (!FnLBBs.empty()) {
1052 const BasicBlock *BBL = FnLBBs.pop_back_val();
1053 const BasicBlock *BBR = FnRBBs.pop_back_val();
Nick Lewyckyf216f69a2010-08-06 07:21:30 +00001054
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001055 if (int Res = cmpValues(BBL, BBR))
1056 return Res;
Nick Lewyckyf216f69a2010-08-06 07:21:30 +00001057
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001058 if (int Res = compare(BBL, BBR))
1059 return Res;
Nick Lewyckyf216f69a2010-08-06 07:21:30 +00001060
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001061 const TerminatorInst *TermL = BBL->getTerminator();
1062 const TerminatorInst *TermR = BBR->getTerminator();
1063
1064 assert(TermL->getNumSuccessors() == TermR->getNumSuccessors());
1065 for (unsigned i = 0, e = TermL->getNumSuccessors(); i != e; ++i) {
David Blaikie70573dc2014-11-19 07:49:26 +00001066 if (!VisitedBBs.insert(TermL->getSuccessor(i)).second)
Nick Lewycky2b3cbac2010-05-13 06:45:13 +00001067 continue;
Nick Lewyckyf216f69a2010-08-06 07:21:30 +00001068
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001069 FnLBBs.push_back(TermL->getSuccessor(i));
1070 FnRBBs.push_back(TermR->getSuccessor(i));
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +00001071 }
1072 }
Stepan Dyatkovskiy17ee5ac2014-06-21 17:55:51 +00001073 return 0;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001074}
1075
Nick Lewycky564fcca2011-01-28 07:36:21 +00001076namespace {
1077
1078/// MergeFunctions finds functions which will generate identical machine code,
1079/// by considering all pointer types to be equivalent. Once identified,
1080/// MergeFunctions will fold them by replacing a call to one to a call to a
1081/// bitcast of the other.
1082///
1083class MergeFunctions : public ModulePass {
1084public:
1085 static char ID;
1086 MergeFunctions()
1087 : ModulePass(ID), HasGlobalAliases(false) {
1088 initializeMergeFunctionsPass(*PassRegistry::getPassRegistry());
1089 }
1090
Craig Topper3e4c6972014-03-05 09:10:37 +00001091 bool runOnModule(Module &M) override;
Nick Lewycky564fcca2011-01-28 07:36:21 +00001092
1093private:
Stepan Dyatkovskiyfe134cd2014-09-10 10:08:25 +00001094 typedef std::set<FunctionNode> FnTreeType;
Nick Lewycky564fcca2011-01-28 07:36:21 +00001095
1096 /// A work queue of functions that may have been modified and should be
1097 /// analyzed again.
1098 std::vector<WeakVH> Deferred;
1099
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +00001100 /// Checks the rules of order relation introduced among functions set.
1101 /// Returns true, if sanity check has been passed, and false if failed.
1102 bool doSanityCheck(std::vector<WeakVH> &Worklist);
1103
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001104 /// Insert a ComparableFunction into the FnTree, or merge it away if it's
Nick Lewycky564fcca2011-01-28 07:36:21 +00001105 /// equal to one that's already present.
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001106 bool insert(Function *NewFunction);
Nick Lewycky564fcca2011-01-28 07:36:21 +00001107
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001108 /// Remove a Function from the FnTree and queue it up for a second sweep of
Nick Lewycky564fcca2011-01-28 07:36:21 +00001109 /// analysis.
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001110 void remove(Function *F);
Nick Lewycky564fcca2011-01-28 07:36:21 +00001111
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001112 /// Find the functions that use this Value and remove them from FnTree and
Nick Lewycky564fcca2011-01-28 07:36:21 +00001113 /// queue the functions.
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001114 void removeUsers(Value *V);
Nick Lewycky564fcca2011-01-28 07:36:21 +00001115
1116 /// Replace all direct calls of Old with calls of New. Will bitcast New if
1117 /// necessary to make types match.
1118 void replaceDirectCallers(Function *Old, Function *New);
1119
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001120 /// Merge two equivalent functions. Upon completion, G may be deleted, or may
1121 /// be converted into a thunk. In either case, it should never be visited
1122 /// again.
1123 void mergeTwoFunctions(Function *F, Function *G);
Nick Lewycky564fcca2011-01-28 07:36:21 +00001124
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001125 /// Replace G with a thunk or an alias to F. Deletes G.
1126 void writeThunkOrAlias(Function *F, Function *G);
Nick Lewycky564fcca2011-01-28 07:36:21 +00001127
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001128 /// Replace G with a simple tail call to bitcast(F). Also replace direct uses
1129 /// of G with bitcast(F). Deletes G.
1130 void writeThunk(Function *F, Function *G);
Nick Lewycky564fcca2011-01-28 07:36:21 +00001131
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001132 /// Replace G with an alias to F. Deletes G.
1133 void writeAlias(Function *F, Function *G);
Nick Lewycky564fcca2011-01-28 07:36:21 +00001134
Arnold Schwaighofer0302da62015-06-09 00:03:29 +00001135 /// Replace function F with function G in the function tree.
1136 void replaceFunctionInTree(FnTreeType::iterator &IterToF, Function *G);
1137
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001138 /// The set of all distinct functions. Use the insert() and remove() methods
1139 /// to modify it.
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001140 FnTreeType FnTree;
Nick Lewycky564fcca2011-01-28 07:36:21 +00001141
Nick Lewycky564fcca2011-01-28 07:36:21 +00001142 /// Whether or not the target supports global aliases.
1143 bool HasGlobalAliases;
1144};
1145
1146} // end anonymous namespace
1147
1148char MergeFunctions::ID = 0;
1149INITIALIZE_PASS(MergeFunctions, "mergefunc", "Merge Functions", false, false)
1150
1151ModulePass *llvm::createMergeFunctionsPass() {
1152 return new MergeFunctions();
1153}
1154
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +00001155bool MergeFunctions::doSanityCheck(std::vector<WeakVH> &Worklist) {
1156 if (const unsigned Max = NumFunctionsForSanityCheck) {
1157 unsigned TripleNumber = 0;
1158 bool Valid = true;
1159
1160 dbgs() << "MERGEFUNC-SANITY: Started for first " << Max << " functions.\n";
1161
1162 unsigned i = 0;
1163 for (std::vector<WeakVH>::iterator I = Worklist.begin(), E = Worklist.end();
1164 I != E && i < Max; ++I, ++i) {
1165 unsigned j = i;
1166 for (std::vector<WeakVH>::iterator J = I; J != E && j < Max; ++J, ++j) {
1167 Function *F1 = cast<Function>(*I);
1168 Function *F2 = cast<Function>(*J);
Mehdi Aminia28d91d2015-03-10 02:37:25 +00001169 int Res1 = FunctionComparator(F1, F2).compare();
1170 int Res2 = FunctionComparator(F2, F1).compare();
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +00001171
1172 // If F1 <= F2, then F2 >= F1, otherwise report failure.
1173 if (Res1 != -Res2) {
1174 dbgs() << "MERGEFUNC-SANITY: Non-symmetric; triple: " << TripleNumber
1175 << "\n";
1176 F1->dump();
1177 F2->dump();
1178 Valid = false;
1179 }
1180
1181 if (Res1 == 0)
1182 continue;
1183
1184 unsigned k = j;
1185 for (std::vector<WeakVH>::iterator K = J; K != E && k < Max;
1186 ++k, ++K, ++TripleNumber) {
1187 if (K == J)
1188 continue;
1189
1190 Function *F3 = cast<Function>(*K);
Mehdi Aminia28d91d2015-03-10 02:37:25 +00001191 int Res3 = FunctionComparator(F1, F3).compare();
1192 int Res4 = FunctionComparator(F2, F3).compare();
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +00001193
1194 bool Transitive = true;
1195
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +00001196 if (Res1 != 0 && Res1 == Res4) {
Stepan Dyatkovskiy0b588012014-06-21 19:07:51 +00001197 // F1 > F2, F2 > F3 => F1 > F3
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +00001198 Transitive = Res3 == Res1;
Stepan Dyatkovskiy0b588012014-06-21 19:07:51 +00001199 } else if (Res3 != 0 && Res3 == -Res4) {
1200 // F1 > F3, F3 > F2 => F1 > F2
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +00001201 Transitive = Res3 == Res1;
Stepan Dyatkovskiy0b588012014-06-21 19:07:51 +00001202 } else if (Res4 != 0 && -Res3 == Res4) {
1203 // F2 > F3, F3 > F1 => F2 > F1
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +00001204 Transitive = Res4 == -Res1;
1205 }
1206
1207 if (!Transitive) {
1208 dbgs() << "MERGEFUNC-SANITY: Non-transitive; triple: "
1209 << TripleNumber << "\n";
1210 dbgs() << "Res1, Res3, Res4: " << Res1 << ", " << Res3 << ", "
1211 << Res4 << "\n";
1212 F1->dump();
1213 F2->dump();
1214 F3->dump();
1215 Valid = false;
1216 }
1217 }
1218 }
1219 }
1220
1221 dbgs() << "MERGEFUNC-SANITY: " << (Valid ? "Passed." : "Failed.") << "\n";
1222 return Valid;
1223 }
1224 return true;
1225}
1226
Nick Lewycky564fcca2011-01-28 07:36:21 +00001227bool MergeFunctions::runOnModule(Module &M) {
1228 bool Changed = false;
Nick Lewycky564fcca2011-01-28 07:36:21 +00001229
1230 for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) {
1231 if (!I->isDeclaration() && !I->hasAvailableExternallyLinkage())
1232 Deferred.push_back(WeakVH(I));
1233 }
Nick Lewycky564fcca2011-01-28 07:36:21 +00001234
1235 do {
1236 std::vector<WeakVH> Worklist;
1237 Deferred.swap(Worklist);
1238
Stepan Dyatkovskiya77f3d82014-06-21 18:58:11 +00001239 DEBUG(doSanityCheck(Worklist));
1240
Nick Lewycky564fcca2011-01-28 07:36:21 +00001241 DEBUG(dbgs() << "size of module: " << M.size() << '\n');
1242 DEBUG(dbgs() << "size of worklist: " << Worklist.size() << '\n');
1243
1244 // Insert only strong functions and merge them. Strong function merging
1245 // always deletes one of them.
1246 for (std::vector<WeakVH>::iterator I = Worklist.begin(),
1247 E = Worklist.end(); I != E; ++I) {
1248 if (!*I) continue;
1249 Function *F = cast<Function>(*I);
1250 if (!F->isDeclaration() && !F->hasAvailableExternallyLinkage() &&
1251 !F->mayBeOverridden()) {
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001252 Changed |= insert(F);
Nick Lewycky564fcca2011-01-28 07:36:21 +00001253 }
1254 }
1255
1256 // Insert only weak functions and merge them. By doing these second we
1257 // create thunks to the strong function when possible. When two weak
1258 // functions are identical, we create a new strong function with two weak
1259 // weak thunks to it which are identical but not mergable.
1260 for (std::vector<WeakVH>::iterator I = Worklist.begin(),
1261 E = Worklist.end(); I != E; ++I) {
1262 if (!*I) continue;
1263 Function *F = cast<Function>(*I);
1264 if (!F->isDeclaration() && !F->hasAvailableExternallyLinkage() &&
1265 F->mayBeOverridden()) {
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001266 Changed |= insert(F);
Nick Lewycky564fcca2011-01-28 07:36:21 +00001267 }
1268 }
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001269 DEBUG(dbgs() << "size of FnTree: " << FnTree.size() << '\n');
Nick Lewycky564fcca2011-01-28 07:36:21 +00001270 } while (!Deferred.empty());
1271
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001272 FnTree.clear();
Nick Lewycky564fcca2011-01-28 07:36:21 +00001273
1274 return Changed;
1275}
1276
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001277// Replace direct callers of Old with New.
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001278void MergeFunctions::replaceDirectCallers(Function *Old, Function *New) {
1279 Constant *BitcastNew = ConstantExpr::getBitCast(New, Old->getType());
Chandler Carruthcdf47882014-03-09 03:16:01 +00001280 for (auto UI = Old->use_begin(), UE = Old->use_end(); UI != UE;) {
1281 Use *U = &*UI;
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001282 ++UI;
Chandler Carruthcdf47882014-03-09 03:16:01 +00001283 CallSite CS(U->getUser());
1284 if (CS && CS.isCallee(U)) {
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001285 remove(CS.getInstruction()->getParent()->getParent());
Chandler Carruthcdf47882014-03-09 03:16:01 +00001286 U->set(BitcastNew);
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001287 }
1288 }
1289}
1290
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001291// Replace G with an alias to F if possible, or else a thunk to F. Deletes G.
1292void MergeFunctions::writeThunkOrAlias(Function *F, Function *G) {
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001293 if (HasGlobalAliases && G->hasUnnamedAddr()) {
1294 if (G->hasExternalLinkage() || G->hasLocalLinkage() ||
1295 G->hasWeakLinkage()) {
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001296 writeAlias(F, G);
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001297 return;
1298 }
1299 }
1300
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001301 writeThunk(F, G);
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001302}
1303
Stepan Dyatkovskiydc2c4b42013-09-17 09:36:11 +00001304// Helper for writeThunk,
1305// Selects proper bitcast operation,
Alp Tokercb402912014-01-24 17:20:08 +00001306// but a bit simpler then CastInst::getCastOpcode.
Carlo Kok307625c2014-04-30 17:53:04 +00001307static Value *createCast(IRBuilder<false> &Builder, Value *V, Type *DestTy) {
Stepan Dyatkovskiydc2c4b42013-09-17 09:36:11 +00001308 Type *SrcTy = V->getType();
Carlo Kok307625c2014-04-30 17:53:04 +00001309 if (SrcTy->isStructTy()) {
1310 assert(DestTy->isStructTy());
1311 assert(SrcTy->getStructNumElements() == DestTy->getStructNumElements());
1312 Value *Result = UndefValue::get(DestTy);
1313 for (unsigned int I = 0, E = SrcTy->getStructNumElements(); I < E; ++I) {
1314 Value *Element = createCast(
Craig Toppere1d12942014-08-27 05:25:25 +00001315 Builder, Builder.CreateExtractValue(V, makeArrayRef(I)),
Carlo Kok307625c2014-04-30 17:53:04 +00001316 DestTy->getStructElementType(I));
1317
1318 Result =
Craig Toppere1d12942014-08-27 05:25:25 +00001319 Builder.CreateInsertValue(Result, Element, makeArrayRef(I));
Carlo Kok307625c2014-04-30 17:53:04 +00001320 }
1321 return Result;
1322 }
1323 assert(!DestTy->isStructTy());
Stepan Dyatkovskiydc2c4b42013-09-17 09:36:11 +00001324 if (SrcTy->isIntegerTy() && DestTy->isPointerTy())
1325 return Builder.CreateIntToPtr(V, DestTy);
1326 else if (SrcTy->isPointerTy() && DestTy->isIntegerTy())
1327 return Builder.CreatePtrToInt(V, DestTy);
1328 else
1329 return Builder.CreateBitCast(V, DestTy);
1330}
1331
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001332// Replace G with a simple tail call to bitcast(F). Also replace direct uses
1333// of G with bitcast(F). Deletes G.
1334void MergeFunctions::writeThunk(Function *F, Function *G) {
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +00001335 if (!G->mayBeOverridden()) {
1336 // Redirect direct callers of G to F.
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001337 replaceDirectCallers(G, F);
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +00001338 }
1339
Nick Lewycky71972d42010-09-07 01:42:10 +00001340 // If G was internal then we may have replaced all uses of G with F. If so,
Nick Lewyckyf216f69a2010-08-06 07:21:30 +00001341 // stop here and delete G. There's no need for a thunk.
1342 if (G->hasLocalLinkage() && G->use_empty()) {
1343 G->eraseFromParent();
1344 return;
1345 }
1346
Nick Lewycky25675ac2009-06-12 15:56:56 +00001347 Function *NewG = Function::Create(G->getFunctionType(), G->getLinkage(), "",
1348 G->getParent());
Owen Anderson55f1c092009-08-13 21:58:54 +00001349 BasicBlock *BB = BasicBlock::Create(F->getContext(), "", NewG);
Nick Lewyckyfbd27572010-08-08 05:04:23 +00001350 IRBuilder<false> Builder(BB);
Nick Lewyckye04dc222009-06-12 08:04:51 +00001351
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +00001352 SmallVector<Value *, 16> Args;
Nick Lewyckye04dc222009-06-12 08:04:51 +00001353 unsigned i = 0;
Chris Lattner229907c2011-07-18 04:54:35 +00001354 FunctionType *FFTy = F->getFunctionType();
Nick Lewyckye04dc222009-06-12 08:04:51 +00001355 for (Function::arg_iterator AI = NewG->arg_begin(), AE = NewG->arg_end();
1356 AI != AE; ++AI) {
Stepan Dyatkovskiydc2c4b42013-09-17 09:36:11 +00001357 Args.push_back(createCast(Builder, (Value*)AI, FFTy->getParamType(i)));
Nick Lewyckye04dc222009-06-12 08:04:51 +00001358 ++i;
1359 }
1360
Jay Foad5bd375a2011-07-15 08:37:34 +00001361 CallInst *CI = Builder.CreateCall(F, Args);
Nick Lewyckye04dc222009-06-12 08:04:51 +00001362 CI->setTailCall();
Nick Lewyckyd5bf51f2009-06-12 16:04:00 +00001363 CI->setCallingConv(F->getCallingConv());
Benjamin Kramerccce8ba2010-01-05 13:12:22 +00001364 if (NewG->getReturnType()->isVoidTy()) {
Nick Lewyckyfbd27572010-08-08 05:04:23 +00001365 Builder.CreateRetVoid();
Nick Lewyckye04dc222009-06-12 08:04:51 +00001366 } else {
Stepan Dyatkovskiydc2c4b42013-09-17 09:36:11 +00001367 Builder.CreateRet(createCast(Builder, CI, NewG->getReturnType()));
Nick Lewyckye04dc222009-06-12 08:04:51 +00001368 }
1369
1370 NewG->copyAttributesFrom(G);
1371 NewG->takeName(G);
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001372 removeUsers(G);
Nick Lewyckye04dc222009-06-12 08:04:51 +00001373 G->replaceAllUsesWith(NewG);
1374 G->eraseFromParent();
Nick Lewycky71972d42010-09-07 01:42:10 +00001375
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001376 DEBUG(dbgs() << "writeThunk: " << NewG->getName() << '\n');
Nick Lewycky71972d42010-09-07 01:42:10 +00001377 ++NumThunksWritten;
Nick Lewyckye04dc222009-06-12 08:04:51 +00001378}
1379
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001380// Replace G with an alias to F and delete G.
1381void MergeFunctions::writeAlias(Function *F, Function *G) {
Rafael Espindola4fe00942014-05-16 13:34:04 +00001382 PointerType *PTy = G->getType();
David Blaikief64246b2015-04-29 21:22:39 +00001383 auto *GA = GlobalAlias::create(PTy, G->getLinkage(), "", F);
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001384 F->setAlignment(std::max(F->getAlignment(), G->getAlignment()));
1385 GA->takeName(G);
1386 GA->setVisibility(G->getVisibility());
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001387 removeUsers(G);
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001388 G->replaceAllUsesWith(GA);
1389 G->eraseFromParent();
1390
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001391 DEBUG(dbgs() << "writeAlias: " << GA->getName() << '\n');
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001392 ++NumAliasesWritten;
1393}
1394
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001395// Merge two equivalent functions. Upon completion, Function G is deleted.
1396void MergeFunctions::mergeTwoFunctions(Function *F, Function *G) {
Nick Lewycky71972d42010-09-07 01:42:10 +00001397 if (F->mayBeOverridden()) {
1398 assert(G->mayBeOverridden());
Nick Lewyckyd3c6dfe2010-05-13 05:48:45 +00001399
Arnold Schwaighofer7e226272015-06-09 18:19:17 +00001400 // Make them both thunks to the same internal function.
1401 Function *H = Function::Create(F->getFunctionType(), F->getLinkage(), "",
1402 F->getParent());
1403 H->copyAttributesFrom(F);
1404 H->takeName(F);
1405 removeUsers(F);
1406 F->replaceAllUsesWith(H);
1407
1408 unsigned MaxAlignment = std::max(G->getAlignment(), H->getAlignment());
1409
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001410 if (HasGlobalAliases) {
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001411 writeAlias(F, G);
1412 writeAlias(F, H);
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001413 } else {
Arnold Schwaighofer7e226272015-06-09 18:19:17 +00001414 writeThunk(F, G);
1415 writeThunk(F, H);
Nick Lewyckyf1cec162011-01-25 08:56:50 +00001416 }
Nick Lewycky71972d42010-09-07 01:42:10 +00001417
Arnold Schwaighofer7e226272015-06-09 18:19:17 +00001418 F->setAlignment(MaxAlignment);
1419 F->setLinkage(GlobalValue::PrivateLinkage);
Nick Lewycky71972d42010-09-07 01:42:10 +00001420 ++NumDoubleWeak;
Nick Lewyckyf216f69a2010-08-06 07:21:30 +00001421 } else {
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001422 writeThunkOrAlias(F, G);
Nick Lewycky3c6d34a2008-11-02 16:46:26 +00001423 }
1424
Nick Lewyckye04dc222009-06-12 08:04:51 +00001425 ++NumFunctionsMerged;
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001426}
1427
Arnold Schwaighofer0302da62015-06-09 00:03:29 +00001428/// Replace function F for function G in the map.
1429void MergeFunctions::replaceFunctionInTree(FnTreeType::iterator &IterToF,
1430 Function *G) {
1431 Function *F = IterToF->getFunc();
1432
1433 // A total order is already guaranteed otherwise because we process strong
1434 // functions before weak functions.
Denis Protivenskyc09e3762015-06-09 09:28:37 +00001435 assert(((F->mayBeOverridden() && G->mayBeOverridden()) ||
1436 (!F->mayBeOverridden() && !G->mayBeOverridden())) &&
Arnold Schwaighofer0302da62015-06-09 00:03:29 +00001437 "Only change functions if both are strong or both are weak");
Arnold Schwaighofer003c2e92015-06-09 00:17:40 +00001438 (void)F;
Arnold Schwaighofer0302da62015-06-09 00:03:29 +00001439
1440 IterToF->replaceBy(G);
1441}
1442
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001443// Insert a ComparableFunction into the FnTree, or merge it away if equal to one
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001444// that was already inserted.
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001445bool MergeFunctions::insert(Function *NewFunction) {
1446 std::pair<FnTreeType::iterator, bool> Result =
Mehdi Aminia28d91d2015-03-10 02:37:25 +00001447 FnTree.insert(FunctionNode(NewFunction));
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001448
Nick Lewycky292e78c2011-02-09 06:32:02 +00001449 if (Result.second) {
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001450 DEBUG(dbgs() << "Inserting as unique: " << NewFunction->getName() << '\n');
Nick Lewycky00959372010-09-05 08:22:49 +00001451 return false;
Nick Lewycky292e78c2011-02-09 06:32:02 +00001452 }
Nick Lewyckyfbd27572010-08-08 05:04:23 +00001453
Stepan Dyatkovskiyfe134cd2014-09-10 10:08:25 +00001454 const FunctionNode &OldF = *Result.first;
Nick Lewycky00959372010-09-05 08:22:49 +00001455
Matt Arsenault517d84e2013-10-01 18:05:30 +00001456 // Don't merge tiny functions, since it can just end up making the function
1457 // larger.
1458 // FIXME: Should still merge them if they are unnamed_addr and produce an
1459 // alias.
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001460 if (NewFunction->size() == 1) {
1461 if (NewFunction->front().size() <= 2) {
1462 DEBUG(dbgs() << NewFunction->getName()
1463 << " is to small to bother merging\n");
Matt Arsenault517d84e2013-10-01 18:05:30 +00001464 return false;
1465 }
1466 }
1467
Arnold Schwaighofer0302da62015-06-09 00:03:29 +00001468 // Impose a total order (by name) on the replacement of functions. This is
1469 // important when operating on more than one module independently to prevent
1470 // cycles of thunks calling each other when the modules are linked together.
1471 //
1472 // When one function is weak and the other is strong there is an order imposed
1473 // already. We process strong functions before weak functions.
1474 if ((OldF.getFunc()->mayBeOverridden() && NewFunction->mayBeOverridden()) ||
1475 (!OldF.getFunc()->mayBeOverridden() && !NewFunction->mayBeOverridden()))
1476 if (OldF.getFunc()->getName() > NewFunction->getName()) {
1477 // Swap the two functions.
1478 Function *F = OldF.getFunc();
1479 replaceFunctionInTree(Result.first, NewFunction);
1480 NewFunction = F;
1481 assert(OldF.getFunc() != F && "Must have swapped the functions.");
1482 }
1483
Nick Lewycky00959372010-09-05 08:22:49 +00001484 // Never thunk a strong function to a weak function.
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001485 assert(!OldF.getFunc()->mayBeOverridden() || NewFunction->mayBeOverridden());
Nick Lewycky00959372010-09-05 08:22:49 +00001486
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001487 DEBUG(dbgs() << " " << OldF.getFunc()->getName()
1488 << " == " << NewFunction->getName() << '\n');
Nick Lewycky00959372010-09-05 08:22:49 +00001489
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001490 Function *DeleteF = NewFunction;
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001491 mergeTwoFunctions(OldF.getFunc(), DeleteF);
Nick Lewycky00959372010-09-05 08:22:49 +00001492 return true;
Nick Lewyckyfbd27572010-08-08 05:04:23 +00001493}
Nick Lewyckyd01d42e2008-11-02 05:52:50 +00001494
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001495// Remove a function from FnTree. If it was already in FnTree, add
1496// it to Deferred so that we'll look at it in the next round.
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001497void MergeFunctions::remove(Function *F) {
Nick Lewycky292e78c2011-02-09 06:32:02 +00001498 // We need to make sure we remove F, not a function "equal" to F per the
1499 // function equality comparator.
Mehdi Aminia28d91d2015-03-10 02:37:25 +00001500 FnTreeType::iterator found = FnTree.find(FunctionNode(F));
Stepan Dyatkovskiyf4af8552014-06-21 20:54:36 +00001501 size_t Erased = 0;
1502 if (found != FnTree.end() && found->getFunc() == F) {
1503 Erased = 1;
1504 FnTree.erase(found);
1505 }
1506
1507 if (Erased) {
1508 DEBUG(dbgs() << "Removed " << F->getName()
1509 << " from set and deferred it.\n");
Benjamin Kramerf5e2fc42015-05-29 19:43:39 +00001510 Deferred.emplace_back(F);
Nick Lewycky0464d1d2010-08-31 05:53:05 +00001511 }
Nick Lewycky4e250c82011-01-02 02:46:33 +00001512}
Nick Lewycky00959372010-09-05 08:22:49 +00001513
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001514// For each instruction used by the value, remove() the function that contains
1515// the instruction. This should happen right before a call to RAUW.
1516void MergeFunctions::removeUsers(Value *V) {
Nick Lewycky5361b842011-01-02 19:16:44 +00001517 std::vector<Value *> Worklist;
1518 Worklist.push_back(V);
1519 while (!Worklist.empty()) {
1520 Value *V = Worklist.back();
1521 Worklist.pop_back();
1522
Chandler Carruthcdf47882014-03-09 03:16:01 +00001523 for (User *U : V->users()) {
1524 if (Instruction *I = dyn_cast<Instruction>(U)) {
Nick Lewyckycfb284c2011-01-28 08:43:14 +00001525 remove(I->getParent()->getParent());
Chandler Carruthcdf47882014-03-09 03:16:01 +00001526 } else if (isa<GlobalValue>(U)) {
Nick Lewycky540f9532011-01-15 10:16:23 +00001527 // do nothing
Chandler Carruthcdf47882014-03-09 03:16:01 +00001528 } else if (Constant *C = dyn_cast<Constant>(U)) {
1529 for (User *UU : C->users())
1530 Worklist.push_back(UU);
Nick Lewycky5361b842011-01-02 19:16:44 +00001531 }
Nick Lewycky00959372010-09-05 08:22:49 +00001532 }
Nick Lewycky0464d1d2010-08-31 05:53:05 +00001533 }
Nick Lewycky00959372010-09-05 08:22:49 +00001534}