0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
1 //===- ExtractFunction.cpp - Extract a function from Program --------------===//
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
2 //
|
147
|
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
|
|
4 // See https://llvm.org/LICENSE.txt for license information.
|
|
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
6 //
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
7 //===----------------------------------------------------------------------===//
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
8 //
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
9 // This file implements several methods that are used to extract functions,
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
10 // loops, or portions of a module from the rest of the module.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
11 //
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
12 //===----------------------------------------------------------------------===//
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
13
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
14 #include "BugDriver.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
15 #include "llvm/IR/Constants.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
16 #include "llvm/IR/DataLayout.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
17 #include "llvm/IR/DerivedTypes.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
18 #include "llvm/IR/LLVMContext.h"
|
83
|
19 #include "llvm/IR/LegacyPassManager.h"
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
20 #include "llvm/IR/Module.h"
|
77
|
21 #include "llvm/IR/Verifier.h"
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
22 #include "llvm/Pass.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
23 #include "llvm/Support/CommandLine.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
24 #include "llvm/Support/Debug.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
25 #include "llvm/Support/FileUtilities.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
26 #include "llvm/Support/Path.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
27 #include "llvm/Support/Signals.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
28 #include "llvm/Support/ToolOutputFile.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
29 #include "llvm/Transforms/IPO.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
30 #include "llvm/Transforms/Scalar.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
31 #include "llvm/Transforms/Utils/Cloning.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
32 #include "llvm/Transforms/Utils/CodeExtractor.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
33 #include <set>
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
34 using namespace llvm;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
35
|
77
|
36 #define DEBUG_TYPE "bugpoint"
|
|
37
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
38 namespace llvm {
|
120
|
39 bool DisableSimplifyCFG = false;
|
|
40 extern cl::opt<std::string> OutputPrefix;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
41 } // End llvm namespace
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
42
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
43 namespace {
|
120
|
44 cl::opt<bool> NoDCE("disable-dce",
|
|
45 cl::desc("Do not use the -dce pass to reduce testcases"));
|
|
46 cl::opt<bool, true>
|
|
47 NoSCFG("disable-simplifycfg", cl::location(DisableSimplifyCFG),
|
|
48 cl::desc("Do not use the -simplifycfg pass to reduce testcases"));
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
49
|
120
|
50 Function *globalInitUsesExternalBA(GlobalVariable *GV) {
|
|
51 if (!GV->hasInitializer())
|
|
52 return nullptr;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
53
|
120
|
54 Constant *I = GV->getInitializer();
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
55
|
120
|
56 // walk the values used by the initializer
|
|
57 // (and recurse into things like ConstantExpr)
|
|
58 std::vector<Constant *> Todo;
|
|
59 std::set<Constant *> Done;
|
|
60 Todo.push_back(I);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
61
|
120
|
62 while (!Todo.empty()) {
|
|
63 Constant *V = Todo.back();
|
|
64 Todo.pop_back();
|
|
65 Done.insert(V);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
66
|
120
|
67 if (BlockAddress *BA = dyn_cast<BlockAddress>(V)) {
|
|
68 Function *F = BA->getFunction();
|
|
69 if (F->isDeclaration())
|
|
70 return F;
|
|
71 }
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
72
|
120
|
73 for (User::op_iterator i = V->op_begin(), e = V->op_end(); i != e; ++i) {
|
|
74 Constant *C = dyn_cast<Constant>(*i);
|
|
75 if (C && !isa<GlobalValue>(C) && !Done.count(C))
|
|
76 Todo.push_back(C);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
77 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
78 }
|
120
|
79 return nullptr;
|
|
80 }
|
|
81 } // end anonymous namespace
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
82
|
77
|
83 std::unique_ptr<Module>
|
|
84 BugDriver::deleteInstructionFromProgram(const Instruction *I,
|
|
85 unsigned Simplification) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
86 // FIXME, use vmap?
|
134
|
87 std::unique_ptr<Module> Clone = CloneModule(*Program);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
88
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
89 const BasicBlock *PBB = I->getParent();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
90 const Function *PF = PBB->getParent();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
91
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
92 Module::iterator RFI = Clone->begin(); // Get iterator to corresponding fn
|
120
|
93 std::advance(
|
|
94 RFI, std::distance(PF->getParent()->begin(), Module::const_iterator(PF)));
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
95
|
120
|
96 Function::iterator RBI = RFI->begin(); // Get iterator to corresponding BB
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
97 std::advance(RBI, std::distance(PF->begin(), Function::const_iterator(PBB)));
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
98
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
99 BasicBlock::iterator RI = RBI->begin(); // Get iterator to corresponding inst
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
100 std::advance(RI, std::distance(PBB->begin(), BasicBlock::const_iterator(I)));
|
100
|
101 Instruction *TheInst = &*RI; // Got the corresponding instruction!
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
102
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
103 // If this instruction produces a value, replace any users with null values
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
104 if (!TheInst->getType()->isVoidTy())
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
105 TheInst->replaceAllUsesWith(Constant::getNullValue(TheInst->getType()));
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
106
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
107 // Remove the instruction from the program.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
108 TheInst->getParent()->getInstList().erase(TheInst);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
109
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
110 // Spiff up the output a little bit.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
111 std::vector<std::string> Passes;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
112
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
113 /// Can we get rid of the -disable-* options?
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
114 if (Simplification > 1 && !NoDCE)
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
115 Passes.push_back("dce");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
116 if (Simplification && !DisableSimplifyCFG)
|
120
|
117 Passes.push_back("simplifycfg"); // Delete dead control flow
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
118
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
119 Passes.push_back("verify");
|
134
|
120 std::unique_ptr<Module> New = runPassesOn(Clone.get(), Passes);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
121 if (!New) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
122 errs() << "Instruction removal failed. Sorry. :( Please report a bug!\n";
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
123 exit(1);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
124 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
125 return New;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
126 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
127
|
77
|
128 std::unique_ptr<Module>
|
147
|
129 BugDriver::performFinalCleanups(std::unique_ptr<Module> M,
|
|
130 bool MayModifySemantics) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
131 // Make all functions external, so GlobalDCE doesn't delete them...
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
132 for (Module::iterator I = M->begin(), E = M->end(); I != E; ++I)
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
133 I->setLinkage(GlobalValue::ExternalLinkage);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
134
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
135 std::vector<std::string> CleanupPasses;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
136 CleanupPasses.push_back("globaldce");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
137
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
138 if (MayModifySemantics)
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
139 CleanupPasses.push_back("deadarghaX0r");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
140 else
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
141 CleanupPasses.push_back("deadargelim");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
142
|
147
|
143 std::unique_ptr<Module> New = runPassesOn(M.get(), CleanupPasses);
|
77
|
144 if (!New) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
145 errs() << "Final cleanups failed. Sorry. :( Please report a bug!\n";
|
77
|
146 return nullptr;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
147 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
148 return New;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
149 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
150
|
77
|
151 std::unique_ptr<Module> BugDriver::extractLoop(Module *M) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
152 std::vector<std::string> LoopExtractPasses;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
153 LoopExtractPasses.push_back("loop-extract-single");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
154
|
77
|
155 std::unique_ptr<Module> NewM = runPassesOn(M, LoopExtractPasses);
|
|
156 if (!NewM) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
157 outs() << "*** Loop extraction failed: ";
|
134
|
158 EmitProgressBitcode(*M, "loopextraction", true);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
159 outs() << "*** Sorry. :( Please report a bug!\n";
|
77
|
160 return nullptr;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
161 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
162
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
163 // Check to see if we created any new functions. If not, no loops were
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
164 // extracted and we should return null. Limit the number of loops we extract
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
165 // to avoid taking forever.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
166 static unsigned NumExtracted = 32;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
167 if (M->size() == NewM->size() || --NumExtracted == 0) {
|
77
|
168 return nullptr;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
169 } else {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
170 assert(M->size() < NewM->size() && "Loop extract removed functions?");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
171 Module::iterator MI = NewM->begin();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
172 for (unsigned i = 0, e = M->size(); i != e; ++i)
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
173 ++MI;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
174 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
175
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
176 return NewM;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
177 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
178
|
100
|
179 static void eliminateAliases(GlobalValue *GV) {
|
|
180 // First, check whether a GlobalAlias references this definition.
|
|
181 // GlobalAlias MAY NOT reference declarations.
|
|
182 for (;;) {
|
|
183 // 1. Find aliases
|
120
|
184 SmallVector<GlobalAlias *, 1> aliases;
|
100
|
185 Module *M = GV->getParent();
|
120
|
186 for (Module::alias_iterator I = M->alias_begin(), E = M->alias_end();
|
|
187 I != E; ++I)
|
100
|
188 if (I->getAliasee()->stripPointerCasts() == GV)
|
|
189 aliases.push_back(&*I);
|
|
190 if (aliases.empty())
|
|
191 break;
|
|
192 // 2. Resolve aliases
|
120
|
193 for (unsigned i = 0, e = aliases.size(); i < e; ++i) {
|
100
|
194 aliases[i]->replaceAllUsesWith(aliases[i]->getAliasee());
|
|
195 aliases[i]->eraseFromParent();
|
|
196 }
|
|
197 // 3. Repeat until no more aliases found; there might
|
|
198 // be an alias to an alias...
|
|
199 }
|
|
200 }
|
|
201
|
|
202 //
|
120
|
203 // DeleteGlobalInitializer - "Remove" the global variable by deleting its
|
|
204 // initializer,
|
100
|
205 // making it external.
|
|
206 //
|
|
207 void llvm::DeleteGlobalInitializer(GlobalVariable *GV) {
|
|
208 eliminateAliases(GV);
|
|
209 GV->setInitializer(nullptr);
|
121
|
210 GV->setComdat(nullptr);
|
100
|
211 }
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
212
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
213 // DeleteFunctionBody - "Remove" the function by deleting all of its basic
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
214 // blocks, making it external.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
215 //
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
216 void llvm::DeleteFunctionBody(Function *F) {
|
100
|
217 eliminateAliases(F);
|
120
|
218 // Function declarations can't have comdats.
|
|
219 F->setComdat(nullptr);
|
100
|
220
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
221 // delete the body of the function...
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
222 F->deleteBody();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
223 assert(F->isDeclaration() && "This didn't make the function external!");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
224 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
225
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
226 /// GetTorInit - Given a list of entries for static ctors/dtors, return them
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
227 /// as a constant array.
|
120
|
228 static Constant *GetTorInit(std::vector<std::pair<Function *, int>> &TorList) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
229 assert(!TorList.empty() && "Don't create empty tor list!");
|
120
|
230 std::vector<Constant *> ArrayElts;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
231 Type *Int32Ty = Type::getInt32Ty(TorList[0].first->getContext());
|
95
|
232
|
121
|
233 StructType *STy = StructType::get(Int32Ty, TorList[0].first->getType());
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
234 for (unsigned i = 0, e = TorList.size(); i != e; ++i) {
|
120
|
235 Constant *Elts[] = {ConstantInt::get(Int32Ty, TorList[i].second),
|
|
236 TorList[i].first};
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
237 ArrayElts.push_back(ConstantStruct::get(STy, Elts));
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
238 }
|
120
|
239 return ConstantArray::get(
|
|
240 ArrayType::get(ArrayElts[0]->getType(), ArrayElts.size()), ArrayElts);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
241 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
242
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
243 /// SplitStaticCtorDtor - A module was recently split into two parts, M1/M2, and
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
244 /// M1 has all of the global variables. If M2 contains any functions that are
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
245 /// static ctors/dtors, we need to add an llvm.global_[cd]tors global to M2, and
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
246 /// prune appropriate entries out of M1s list.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
247 static void SplitStaticCtorDtor(const char *GlobalName, Module *M1, Module *M2,
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
248 ValueToValueMapTy &VMap) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
249 GlobalVariable *GV = M1->getNamedGlobal(GlobalName);
|
120
|
250 if (!GV || GV->isDeclaration() || GV->hasLocalLinkage() || !GV->use_empty())
|
|
251 return;
|
|
252
|
|
253 std::vector<std::pair<Function *, int>> M1Tors, M2Tors;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
254 ConstantArray *InitList = dyn_cast<ConstantArray>(GV->getInitializer());
|
120
|
255 if (!InitList)
|
|
256 return;
|
|
257
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
258 for (unsigned i = 0, e = InitList->getNumOperands(); i != e; ++i) {
|
120
|
259 if (ConstantStruct *CS =
|
|
260 dyn_cast<ConstantStruct>(InitList->getOperand(i))) {
|
|
261 if (CS->getNumOperands() != 2)
|
|
262 return; // Not array of 2-element structs.
|
|
263
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
264 if (CS->getOperand(1)->isNullValue())
|
120
|
265 break; // Found a null terminator, stop here.
|
|
266
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
267 ConstantInt *CI = dyn_cast<ConstantInt>(CS->getOperand(0));
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
268 int Priority = CI ? CI->getSExtValue() : 0;
|
120
|
269
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
270 Constant *FP = CS->getOperand(1);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
271 if (ConstantExpr *CE = dyn_cast<ConstantExpr>(FP))
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
272 if (CE->isCast())
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
273 FP = CE->getOperand(0);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
274 if (Function *F = dyn_cast<Function>(FP)) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
275 if (!F->isDeclaration())
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
276 M1Tors.push_back(std::make_pair(F, Priority));
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
277 else {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
278 // Map to M2's version of the function.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
279 F = cast<Function>(VMap[F]);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
280 M2Tors.push_back(std::make_pair(F, Priority));
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
281 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
282 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
283 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
284 }
|
120
|
285
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
286 GV->eraseFromParent();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
287 if (!M1Tors.empty()) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
288 Constant *M1Init = GetTorInit(M1Tors);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
289 new GlobalVariable(*M1, M1Init->getType(), false,
|
120
|
290 GlobalValue::AppendingLinkage, M1Init, GlobalName);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
291 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
292
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
293 GV = M2->getNamedGlobal(GlobalName);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
294 assert(GV && "Not a clone of M1?");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
295 assert(GV->use_empty() && "llvm.ctors shouldn't have uses!");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
296
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
297 GV->eraseFromParent();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
298 if (!M2Tors.empty()) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
299 Constant *M2Init = GetTorInit(M2Tors);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
300 new GlobalVariable(*M2, M2Init->getType(), false,
|
120
|
301 GlobalValue::AppendingLinkage, M2Init, GlobalName);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
302 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
303 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
304
|
100
|
305 std::unique_ptr<Module>
|
|
306 llvm::SplitFunctionsOutOfModule(Module *M, const std::vector<Function *> &F,
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
307 ValueToValueMapTy &VMap) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
308 // Make sure functions & globals are all external so that linkage
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
309 // between the two modules will work.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
310 for (Module::iterator I = M->begin(), E = M->end(); I != E; ++I)
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
311 I->setLinkage(GlobalValue::ExternalLinkage);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
312 for (Module::global_iterator I = M->global_begin(), E = M->global_end();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
313 I != E; ++I) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
314 if (I->hasName() && I->getName()[0] == '\01')
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
315 I->setName(I->getName().substr(1));
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
316 I->setLinkage(GlobalValue::ExternalLinkage);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
317 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
318
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
319 ValueToValueMapTy NewVMap;
|
134
|
320 std::unique_ptr<Module> New = CloneModule(*M, NewVMap);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
321
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
322 // Remove the Test functions from the Safe module
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
323 std::set<Function *> TestFunctions;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
324 for (unsigned i = 0, e = F.size(); i != e; ++i) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
325 Function *TNOF = cast<Function>(VMap[F[i]]);
|
147
|
326 LLVM_DEBUG(errs() << "Removing function ");
|
|
327 LLVM_DEBUG(TNOF->printAsOperand(errs(), false));
|
|
328 LLVM_DEBUG(errs() << "\n");
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
329 TestFunctions.insert(cast<Function>(NewVMap[TNOF]));
|
120
|
330 DeleteFunctionBody(TNOF); // Function is now external in this module!
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
331 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
332
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
333 // Remove the Safe functions from the Test module
|
100
|
334 for (Function &I : *New)
|
|
335 if (!TestFunctions.count(&I))
|
|
336 DeleteFunctionBody(&I);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
337
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
338 // Try to split the global initializers evenly
|
100
|
339 for (GlobalVariable &I : M->globals()) {
|
|
340 GlobalVariable *GV = cast<GlobalVariable>(NewVMap[&I]);
|
|
341 if (Function *TestFn = globalInitUsesExternalBA(&I)) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
342 if (Function *SafeFn = globalInitUsesExternalBA(GV)) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
343 errs() << "*** Error: when reducing functions, encountered "
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
344 "the global '";
|
77
|
345 GV->printAsOperand(errs(), false);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
346 errs() << "' with an initializer that references blockaddresses "
|
120
|
347 "from safe function '"
|
|
348 << SafeFn->getName() << "' and from test function '"
|
|
349 << TestFn->getName() << "'.\n";
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
350 exit(1);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
351 }
|
100
|
352 DeleteGlobalInitializer(&I); // Delete the initializer to make it external
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
353 } else {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
354 // If we keep it in the safe module, then delete it in the test module
|
100
|
355 DeleteGlobalInitializer(GV);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
356 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
357 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
358
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
359 // Make sure that there is a global ctor/dtor array in both halves of the
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
360 // module if they both have static ctor/dtor functions.
|
100
|
361 SplitStaticCtorDtor("llvm.global_ctors", M, New.get(), NewVMap);
|
|
362 SplitStaticCtorDtor("llvm.global_dtors", M, New.get(), NewVMap);
|
|
363
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
364 return New;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
365 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
366
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
367 //===----------------------------------------------------------------------===//
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
368 // Basic Block Extraction Code
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
369 //===----------------------------------------------------------------------===//
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
370
|
77
|
371 std::unique_ptr<Module>
|
|
372 BugDriver::extractMappedBlocksFromModule(const std::vector<BasicBlock *> &BBs,
|
|
373 Module *M) {
|
134
|
374 auto Temp = sys::fs::TempFile::create(OutputPrefix + "-extractblocks%%%%%%%");
|
|
375 if (!Temp) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
376 outs() << "*** Basic Block extraction failed!\n";
|
134
|
377 errs() << "Error creating temporary file: " << toString(Temp.takeError())
|
|
378 << "\n";
|
|
379 EmitProgressBitcode(*M, "basicblockextractfail", true);
|
77
|
380 return nullptr;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
381 }
|
134
|
382 DiscardTemp Discard{*Temp};
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
383
|
134
|
384 // Extract all of the blocks except the ones in BBs.
|
|
385 SmallVector<BasicBlock *, 32> BlocksToExtract;
|
|
386 for (Function &F : *M)
|
|
387 for (BasicBlock &BB : F)
|
|
388 // Check if this block is going to be extracted.
|
|
389 if (std::find(BBs.begin(), BBs.end(), &BB) == BBs.end())
|
|
390 BlocksToExtract.push_back(&BB);
|
|
391
|
|
392 raw_fd_ostream OS(Temp->FD, /*shouldClose*/ false);
|
|
393 for (BasicBlock *BB : BBs) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
394 // If the BB doesn't have a name, give it one so we have something to key
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
395 // off of.
|
120
|
396 if (!BB->hasName())
|
|
397 BB->setName("tmpbb");
|
134
|
398 OS << BB->getParent()->getName() << " " << BB->getName() << "\n";
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
399 }
|
134
|
400 OS.flush();
|
|
401 if (OS.has_error()) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
402 errs() << "Error writing list of blocks to not extract\n";
|
134
|
403 EmitProgressBitcode(*M, "basicblockextractfail", true);
|
|
404 OS.clear_error();
|
77
|
405 return nullptr;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
406 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
407
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
408 std::string uniqueFN = "--extract-blocks-file=";
|
134
|
409 uniqueFN += Temp->TmpName;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
410 const char *ExtraArg = uniqueFN.c_str();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
411
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
412 std::vector<std::string> PI;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
413 PI.push_back("extract-blocks");
|
120
|
414 std::unique_ptr<Module> Ret = runPassesOn(M, PI, 1, &ExtraArg);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
415
|
77
|
416 if (!Ret) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
417 outs() << "*** Basic Block extraction failed, please report a bug!\n";
|
134
|
418 EmitProgressBitcode(*M, "basicblockextractfail", true);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
419 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
420 return Ret;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
421 }
|