1 //===- Miscompilation.cpp - Debug program miscompilations -----------------===//
3 // The LLVM Compiler Infrastructure
5 // This file is distributed under the University of Illinois Open Source
6 // License. See LICENSE.TXT for details.
8 //===----------------------------------------------------------------------===//
10 // This file implements optimizer and code generation miscompilation debugging
13 //===----------------------------------------------------------------------===//
15 #include "BugDriver.h"
16 #include "ListReducer.h"
17 #include "ToolRunner.h"
18 #include "llvm/Constants.h"
19 #include "llvm/DerivedTypes.h"
20 #include "llvm/Instructions.h"
21 #include "llvm/Linker.h"
22 #include "llvm/Module.h"
23 #include "llvm/Pass.h"
24 #include "llvm/Analysis/Verifier.h"
25 #include "llvm/Transforms/Utils/Cloning.h"
26 #include "llvm/Support/CommandLine.h"
27 #include "llvm/Support/FileUtilities.h"
28 #include "llvm/Config/config.h" // for HAVE_LINK_R
32 extern cl::opt
<std::string
> OutputPrefix
;
33 extern cl::list
<std::string
> InputArgv
;
37 static llvm::cl::opt
<bool>
38 DisableLoopExtraction("disable-loop-extraction",
39 cl::desc("Don't extract loops when searching for miscompilations"),
41 static llvm::cl::opt
<bool>
42 DisableBlockExtraction("disable-block-extraction",
43 cl::desc("Don't extract blocks when searching for miscompilations"),
46 class ReduceMiscompilingPasses
: public ListReducer
<std::string
> {
49 ReduceMiscompilingPasses(BugDriver
&bd
) : BD(bd
) {}
51 virtual TestResult
doTest(std::vector
<std::string
> &Prefix
,
52 std::vector
<std::string
> &Suffix
,
57 /// TestResult - After passes have been split into a test group and a control
58 /// group, see if they still break the program.
60 ReduceMiscompilingPasses::TestResult
61 ReduceMiscompilingPasses::doTest(std::vector
<std::string
> &Prefix
,
62 std::vector
<std::string
> &Suffix
,
64 // First, run the program with just the Suffix passes. If it is still broken
65 // with JUST the kept passes, discard the prefix passes.
66 outs() << "Checking to see if '" << getPassesString(Suffix
)
67 << "' compiles correctly: ";
69 std::string BitcodeResult
;
70 if (BD
.runPasses(BD
.getProgram(), Suffix
, BitcodeResult
, false/*delete*/,
72 errs() << " Error running this sequence of passes"
73 << " on the input program!\n";
74 BD
.setPassesToRun(Suffix
);
75 BD
.EmitProgressBitcode(BD
.getProgram(), "pass-error", false);
76 exit(BD
.debugOptimizerCrash());
79 // Check to see if the finished program matches the reference output...
80 bool Diff
= BD
.diffProgram(BD
.getProgram(), BitcodeResult
, "",
81 true /*delete bitcode*/, &Error
);
87 errs() << BD
.getToolName() << ": I'm confused: the test fails when "
88 << "no passes are run, nondeterministic program?\n";
91 return KeepSuffix
; // Miscompilation detected!
93 outs() << " yup.\n"; // No miscompilation!
95 if (Prefix
.empty()) return NoFailure
;
97 // Next, see if the program is broken if we run the "prefix" passes first,
98 // then separately run the "kept" passes.
99 outs() << "Checking to see if '" << getPassesString(Prefix
)
100 << "' compiles correctly: ";
102 // If it is not broken with the kept passes, it's possible that the prefix
103 // passes must be run before the kept passes to break it. If the program
104 // WORKS after the prefix passes, but then fails if running the prefix AND
105 // kept passes, we can update our bitcode file to include the result of the
106 // prefix passes, then discard the prefix passes.
108 if (BD
.runPasses(BD
.getProgram(), Prefix
, BitcodeResult
, false/*delete*/,
110 errs() << " Error running this sequence of passes"
111 << " on the input program!\n";
112 BD
.setPassesToRun(Prefix
);
113 BD
.EmitProgressBitcode(BD
.getProgram(), "pass-error", false);
114 exit(BD
.debugOptimizerCrash());
117 // If the prefix maintains the predicate by itself, only keep the prefix!
118 Diff
= BD
.diffProgram(BD
.getProgram(), BitcodeResult
, "", false, &Error
);
120 return InternalError
;
122 outs() << " nope.\n";
123 sys::Path(BitcodeResult
).eraseFromDisk();
126 outs() << " yup.\n"; // No miscompilation!
128 // Ok, so now we know that the prefix passes work, try running the suffix
129 // passes on the result of the prefix passes.
131 OwningPtr
<Module
> PrefixOutput(ParseInputFile(BitcodeResult
,
133 if (PrefixOutput
== 0) {
134 errs() << BD
.getToolName() << ": Error reading bitcode file '"
135 << BitcodeResult
<< "'!\n";
138 sys::Path(BitcodeResult
).eraseFromDisk(); // No longer need the file on disk
140 // Don't check if there are no passes in the suffix.
144 outs() << "Checking to see if '" << getPassesString(Suffix
)
145 << "' passes compile correctly after the '"
146 << getPassesString(Prefix
) << "' passes: ";
148 OwningPtr
<Module
> OriginalInput(BD
.swapProgramIn(PrefixOutput
.take()));
149 if (BD
.runPasses(BD
.getProgram(), Suffix
, BitcodeResult
, false/*delete*/,
151 errs() << " Error running this sequence of passes"
152 << " on the input program!\n";
153 BD
.setPassesToRun(Suffix
);
154 BD
.EmitProgressBitcode(BD
.getProgram(), "pass-error", false);
155 exit(BD
.debugOptimizerCrash());
159 Diff
= BD
.diffProgram(BD
.getProgram(), BitcodeResult
, "",
160 true /*delete bitcode*/, &Error
);
162 return InternalError
;
164 outs() << " nope.\n";
168 // Otherwise, we must not be running the bad pass anymore.
169 outs() << " yup.\n"; // No miscompilation!
170 // Restore orig program & free test.
171 delete BD
.swapProgramIn(OriginalInput
.take());
176 class ReduceMiscompilingFunctions
: public ListReducer
<Function
*> {
178 bool (*TestFn
)(BugDriver
&, Module
*, Module
*, std::string
&);
180 ReduceMiscompilingFunctions(BugDriver
&bd
,
181 bool (*F
)(BugDriver
&, Module
*, Module
*,
183 : BD(bd
), TestFn(F
) {}
185 virtual TestResult
doTest(std::vector
<Function
*> &Prefix
,
186 std::vector
<Function
*> &Suffix
,
187 std::string
&Error
) {
188 if (!Suffix
.empty()) {
189 bool Ret
= TestFuncs(Suffix
, Error
);
191 return InternalError
;
195 if (!Prefix
.empty()) {
196 bool Ret
= TestFuncs(Prefix
, Error
);
198 return InternalError
;
205 bool TestFuncs(const std::vector
<Function
*> &Prefix
, std::string
&Error
);
209 /// TestMergedProgram - Given two modules, link them together and run the
210 /// program, checking to see if the program matches the diff. If there is
211 /// an error, return NULL. If not, return the merged module. The Broken argument
212 /// will be set to true if the output is different. If the DeleteInputs
213 /// argument is set to true then this function deletes both input
214 /// modules before it returns.
216 static Module
*TestMergedProgram(const BugDriver
&BD
, Module
*M1
, Module
*M2
,
217 bool DeleteInputs
, std::string
&Error
,
219 // Link the two portions of the program back to together.
220 std::string ErrorMsg
;
222 M1
= CloneModule(M1
);
223 M2
= CloneModule(M2
);
225 if (Linker::LinkModules(M1
, M2
, &ErrorMsg
)) {
226 errs() << BD
.getToolName() << ": Error linking modules together:"
230 delete M2
; // We are done with this module.
232 // Execute the program.
233 Broken
= BD
.diffProgram(M1
, "", "", false, &Error
);
234 if (!Error
.empty()) {
235 // Delete the linked module
242 /// TestFuncs - split functions in a Module into two groups: those that are
243 /// under consideration for miscompilation vs. those that are not, and test
244 /// accordingly. Each group of functions becomes a separate Module.
246 bool ReduceMiscompilingFunctions::TestFuncs(const std::vector
<Function
*> &Funcs
,
247 std::string
&Error
) {
248 // Test to see if the function is misoptimized if we ONLY run it on the
249 // functions listed in Funcs.
250 outs() << "Checking to see if the program is misoptimized when "
251 << (Funcs
.size()==1 ? "this function is" : "these functions are")
252 << " run through the pass"
253 << (BD
.getPassesToRun().size() == 1 ? "" : "es") << ":";
254 PrintFunctionList(Funcs
);
257 // Create a clone for two reasons:
258 // * If the optimization passes delete any function, the deleted function
259 // will be in the clone and Funcs will still point to valid memory
260 // * If the optimization passes use interprocedural information to break
261 // a function, we want to continue with the original function. Otherwise
262 // we can conclude that a function triggers the bug when in fact one
263 // needs a larger set of original functions to do so.
264 ValueToValueMapTy VMap
;
265 Module
*Clone
= CloneModule(BD
.getProgram(), VMap
);
266 Module
*Orig
= BD
.swapProgramIn(Clone
);
268 std::vector
<Function
*> FuncsOnClone
;
269 for (unsigned i
= 0, e
= Funcs
.size(); i
!= e
; ++i
) {
270 Function
*F
= cast
<Function
>(VMap
[Funcs
[i
]]);
271 FuncsOnClone
.push_back(F
);
274 // Split the module into the two halves of the program we want.
276 Module
*ToNotOptimize
= CloneModule(BD
.getProgram(), VMap
);
277 Module
*ToOptimize
= SplitFunctionsOutOfModule(ToNotOptimize
, FuncsOnClone
,
280 // Run the predicate, note that the predicate will delete both input modules.
281 bool Broken
= TestFn(BD
, ToOptimize
, ToNotOptimize
, Error
);
283 delete BD
.swapProgramIn(Orig
);
288 /// DisambiguateGlobalSymbols - Give anonymous global values names.
290 static void DisambiguateGlobalSymbols(Module
*M
) {
291 for (Module::global_iterator I
= M
->global_begin(), E
= M
->global_end();
294 I
->setName("anon_global");
295 for (Module::iterator I
= M
->begin(), E
= M
->end(); I
!= E
; ++I
)
297 I
->setName("anon_fn");
300 /// ExtractLoops - Given a reduced list of functions that still exposed the bug,
301 /// check to see if we can extract the loops in the region without obscuring the
302 /// bug. If so, it reduces the amount of code identified.
304 static bool ExtractLoops(BugDriver
&BD
,
305 bool (*TestFn
)(BugDriver
&, Module
*, Module
*,
307 std::vector
<Function
*> &MiscompiledFunctions
,
308 std::string
&Error
) {
309 bool MadeChange
= false;
311 if (BugpointIsInterrupted
) return MadeChange
;
313 ValueToValueMapTy VMap
;
314 Module
*ToNotOptimize
= CloneModule(BD
.getProgram(), VMap
);
315 Module
*ToOptimize
= SplitFunctionsOutOfModule(ToNotOptimize
,
316 MiscompiledFunctions
,
318 Module
*ToOptimizeLoopExtracted
= BD
.ExtractLoop(ToOptimize
);
319 if (!ToOptimizeLoopExtracted
) {
320 // If the loop extractor crashed or if there were no extractible loops,
321 // then this chapter of our odyssey is over with.
322 delete ToNotOptimize
;
327 errs() << "Extracted a loop from the breaking portion of the program.\n";
329 // Bugpoint is intentionally not very trusting of LLVM transformations. In
330 // particular, we're not going to assume that the loop extractor works, so
331 // we're going to test the newly loop extracted program to make sure nothing
332 // has broken. If something broke, then we'll inform the user and stop
334 AbstractInterpreter
*AI
= BD
.switchToSafeInterpreter();
336 Module
*New
= TestMergedProgram(BD
, ToOptimizeLoopExtracted
, ToNotOptimize
,
337 false, Error
, Failure
);
340 // Delete the original and set the new program.
341 delete BD
.swapProgramIn(New
);
343 BD
.switchToInterpreter(AI
);
345 // Merged program doesn't work anymore!
346 errs() << " *** ERROR: Loop extraction broke the program. :("
347 << " Please report a bug!\n";
348 errs() << " Continuing on with un-loop-extracted version.\n";
350 BD
.writeProgramToFile(OutputPrefix
+ "-loop-extract-fail-tno.bc",
352 BD
.writeProgramToFile(OutputPrefix
+ "-loop-extract-fail-to.bc",
354 BD
.writeProgramToFile(OutputPrefix
+ "-loop-extract-fail-to-le.bc",
355 ToOptimizeLoopExtracted
);
357 errs() << "Please submit the "
358 << OutputPrefix
<< "-loop-extract-fail-*.bc files.\n";
360 delete ToNotOptimize
;
361 delete ToOptimizeLoopExtracted
;
365 BD
.switchToInterpreter(AI
);
367 outs() << " Testing after loop extraction:\n";
368 // Clone modules, the tester function will free them.
369 Module
*TOLEBackup
= CloneModule(ToOptimizeLoopExtracted
);
370 Module
*TNOBackup
= CloneModule(ToNotOptimize
);
371 Failure
= TestFn(BD
, ToOptimizeLoopExtracted
, ToNotOptimize
, Error
);
375 outs() << "*** Loop extraction masked the problem. Undoing.\n";
376 // If the program is not still broken, then loop extraction did something
377 // that masked the error. Stop loop extraction now.
382 ToOptimizeLoopExtracted
= TOLEBackup
;
383 ToNotOptimize
= TNOBackup
;
385 outs() << "*** Loop extraction successful!\n";
387 std::vector
<std::pair
<std::string
, const FunctionType
*> > MisCompFunctions
;
388 for (Module::iterator I
= ToOptimizeLoopExtracted
->begin(),
389 E
= ToOptimizeLoopExtracted
->end(); I
!= E
; ++I
)
390 if (!I
->isDeclaration())
391 MisCompFunctions
.push_back(std::make_pair(I
->getName(),
392 I
->getFunctionType()));
394 // Okay, great! Now we know that we extracted a loop and that loop
395 // extraction both didn't break the program, and didn't mask the problem.
396 // Replace the current program with the loop extracted version, and try to
397 // extract another loop.
398 std::string ErrorMsg
;
399 if (Linker::LinkModules(ToNotOptimize
, ToOptimizeLoopExtracted
, &ErrorMsg
)){
400 errs() << BD
.getToolName() << ": Error linking modules together:"
404 delete ToOptimizeLoopExtracted
;
406 // All of the Function*'s in the MiscompiledFunctions list are in the old
407 // module. Update this list to include all of the functions in the
408 // optimized and loop extracted module.
409 MiscompiledFunctions
.clear();
410 for (unsigned i
= 0, e
= MisCompFunctions
.size(); i
!= e
; ++i
) {
411 Function
*NewF
= ToNotOptimize
->getFunction(MisCompFunctions
[i
].first
);
413 assert(NewF
&& "Function not found??");
414 assert(NewF
->getFunctionType() == MisCompFunctions
[i
].second
&&
415 "found wrong function type?");
416 MiscompiledFunctions
.push_back(NewF
);
419 BD
.setNewProgram(ToNotOptimize
);
425 class ReduceMiscompiledBlocks
: public ListReducer
<BasicBlock
*> {
427 bool (*TestFn
)(BugDriver
&, Module
*, Module
*, std::string
&);
428 std::vector
<Function
*> FunctionsBeingTested
;
430 ReduceMiscompiledBlocks(BugDriver
&bd
,
431 bool (*F
)(BugDriver
&, Module
*, Module
*,
433 const std::vector
<Function
*> &Fns
)
434 : BD(bd
), TestFn(F
), FunctionsBeingTested(Fns
) {}
436 virtual TestResult
doTest(std::vector
<BasicBlock
*> &Prefix
,
437 std::vector
<BasicBlock
*> &Suffix
,
438 std::string
&Error
) {
439 if (!Suffix
.empty()) {
440 bool Ret
= TestFuncs(Suffix
, Error
);
442 return InternalError
;
446 if (!Prefix
.empty()) {
447 bool Ret
= TestFuncs(Prefix
, Error
);
449 return InternalError
;
456 bool TestFuncs(const std::vector
<BasicBlock
*> &BBs
, std::string
&Error
);
460 /// TestFuncs - Extract all blocks for the miscompiled functions except for the
461 /// specified blocks. If the problem still exists, return true.
463 bool ReduceMiscompiledBlocks::TestFuncs(const std::vector
<BasicBlock
*> &BBs
,
464 std::string
&Error
) {
465 // Test to see if the function is misoptimized if we ONLY run it on the
466 // functions listed in Funcs.
467 outs() << "Checking to see if the program is misoptimized when all ";
469 outs() << "but these " << BBs
.size() << " blocks are extracted: ";
470 for (unsigned i
= 0, e
= BBs
.size() < 10 ? BBs
.size() : 10; i
!= e
; ++i
)
471 outs() << BBs
[i
]->getName() << " ";
472 if (BBs
.size() > 10) outs() << "...";
474 outs() << "blocks are extracted.";
478 // Split the module into the two halves of the program we want.
479 ValueToValueMapTy VMap
;
480 Module
*Clone
= CloneModule(BD
.getProgram(), VMap
);
481 Module
*Orig
= BD
.swapProgramIn(Clone
);
482 std::vector
<Function
*> FuncsOnClone
;
483 std::vector
<BasicBlock
*> BBsOnClone
;
484 for (unsigned i
= 0, e
= FunctionsBeingTested
.size(); i
!= e
; ++i
) {
485 Function
*F
= cast
<Function
>(VMap
[FunctionsBeingTested
[i
]]);
486 FuncsOnClone
.push_back(F
);
488 for (unsigned i
= 0, e
= BBs
.size(); i
!= e
; ++i
) {
489 BasicBlock
*BB
= cast
<BasicBlock
>(VMap
[BBs
[i
]]);
490 BBsOnClone
.push_back(BB
);
494 Module
*ToNotOptimize
= CloneModule(BD
.getProgram(), VMap
);
495 Module
*ToOptimize
= SplitFunctionsOutOfModule(ToNotOptimize
,
499 // Try the extraction. If it doesn't work, then the block extractor crashed
500 // or something, in which case bugpoint can't chase down this possibility.
501 if (Module
*New
= BD
.ExtractMappedBlocksFromModule(BBsOnClone
, ToOptimize
)) {
503 // Run the predicate,
504 // note that the predicate will delete both input modules.
505 bool Ret
= TestFn(BD
, New
, ToNotOptimize
, Error
);
506 delete BD
.swapProgramIn(Orig
);
509 delete BD
.swapProgramIn(Orig
);
511 delete ToNotOptimize
;
516 /// ExtractBlocks - Given a reduced list of functions that still expose the bug,
517 /// extract as many basic blocks from the region as possible without obscuring
520 static bool ExtractBlocks(BugDriver
&BD
,
521 bool (*TestFn
)(BugDriver
&, Module
*, Module
*,
523 std::vector
<Function
*> &MiscompiledFunctions
,
524 std::string
&Error
) {
525 if (BugpointIsInterrupted
) return false;
527 std::vector
<BasicBlock
*> Blocks
;
528 for (unsigned i
= 0, e
= MiscompiledFunctions
.size(); i
!= e
; ++i
)
529 for (Function::iterator I
= MiscompiledFunctions
[i
]->begin(),
530 E
= MiscompiledFunctions
[i
]->end(); I
!= E
; ++I
)
533 // Use the list reducer to identify blocks that can be extracted without
534 // obscuring the bug. The Blocks list will end up containing blocks that must
535 // be retained from the original program.
536 unsigned OldSize
= Blocks
.size();
538 // Check to see if all blocks are extractible first.
539 bool Ret
= ReduceMiscompiledBlocks(BD
, TestFn
, MiscompiledFunctions
)
540 .TestFuncs(std::vector
<BasicBlock
*>(), Error
);
546 ReduceMiscompiledBlocks(BD
, TestFn
,
547 MiscompiledFunctions
).reduceList(Blocks
, Error
);
550 if (Blocks
.size() == OldSize
)
554 ValueToValueMapTy VMap
;
555 Module
*ProgClone
= CloneModule(BD
.getProgram(), VMap
);
556 Module
*ToExtract
= SplitFunctionsOutOfModule(ProgClone
,
557 MiscompiledFunctions
,
559 Module
*Extracted
= BD
.ExtractMappedBlocksFromModule(Blocks
, ToExtract
);
560 if (Extracted
== 0) {
561 // Weird, extraction should have worked.
562 errs() << "Nondeterministic problem extracting blocks??\n";
568 // Otherwise, block extraction succeeded. Link the two program fragments back
572 std::vector
<std::pair
<std::string
, const FunctionType
*> > MisCompFunctions
;
573 for (Module::iterator I
= Extracted
->begin(), E
= Extracted
->end();
575 if (!I
->isDeclaration())
576 MisCompFunctions
.push_back(std::make_pair(I
->getName(),
577 I
->getFunctionType()));
579 std::string ErrorMsg
;
580 if (Linker::LinkModules(ProgClone
, Extracted
, &ErrorMsg
)) {
581 errs() << BD
.getToolName() << ": Error linking modules together:"
587 // Set the new program and delete the old one.
588 BD
.setNewProgram(ProgClone
);
590 // Update the list of miscompiled functions.
591 MiscompiledFunctions
.clear();
593 for (unsigned i
= 0, e
= MisCompFunctions
.size(); i
!= e
; ++i
) {
594 Function
*NewF
= ProgClone
->getFunction(MisCompFunctions
[i
].first
);
595 assert(NewF
&& "Function not found??");
596 assert(NewF
->getFunctionType() == MisCompFunctions
[i
].second
&&
597 "Function has wrong type??");
598 MiscompiledFunctions
.push_back(NewF
);
605 /// DebugAMiscompilation - This is a generic driver to narrow down
606 /// miscompilations, either in an optimization or a code generator.
608 static std::vector
<Function
*>
609 DebugAMiscompilation(BugDriver
&BD
,
610 bool (*TestFn
)(BugDriver
&, Module
*, Module
*,
612 std::string
&Error
) {
613 // Okay, now that we have reduced the list of passes which are causing the
614 // failure, see if we can pin down which functions are being
615 // miscompiled... first build a list of all of the non-external functions in
617 std::vector
<Function
*> MiscompiledFunctions
;
618 Module
*Prog
= BD
.getProgram();
619 for (Module::iterator I
= Prog
->begin(), E
= Prog
->end(); I
!= E
; ++I
)
620 if (!I
->isDeclaration())
621 MiscompiledFunctions
.push_back(I
);
623 // Do the reduction...
624 if (!BugpointIsInterrupted
)
625 ReduceMiscompilingFunctions(BD
, TestFn
).reduceList(MiscompiledFunctions
,
627 if (!Error
.empty()) {
628 errs() << "\n***Cannot reduce functions: ";
629 return MiscompiledFunctions
;
631 outs() << "\n*** The following function"
632 << (MiscompiledFunctions
.size() == 1 ? " is" : "s are")
633 << " being miscompiled: ";
634 PrintFunctionList(MiscompiledFunctions
);
637 // See if we can rip any loops out of the miscompiled functions and still
638 // trigger the problem.
640 if (!BugpointIsInterrupted
&& !DisableLoopExtraction
) {
641 bool Ret
= ExtractLoops(BD
, TestFn
, MiscompiledFunctions
, Error
);
643 return MiscompiledFunctions
;
645 // Okay, we extracted some loops and the problem still appears. See if
646 // we can eliminate some of the created functions from being candidates.
647 DisambiguateGlobalSymbols(BD
.getProgram());
649 // Do the reduction...
650 if (!BugpointIsInterrupted
)
651 ReduceMiscompilingFunctions(BD
, TestFn
).reduceList(MiscompiledFunctions
,
654 return MiscompiledFunctions
;
656 outs() << "\n*** The following function"
657 << (MiscompiledFunctions
.size() == 1 ? " is" : "s are")
658 << " being miscompiled: ";
659 PrintFunctionList(MiscompiledFunctions
);
664 if (!BugpointIsInterrupted
&& !DisableBlockExtraction
) {
665 bool Ret
= ExtractBlocks(BD
, TestFn
, MiscompiledFunctions
, Error
);
667 return MiscompiledFunctions
;
669 // Okay, we extracted some blocks and the problem still appears. See if
670 // we can eliminate some of the created functions from being candidates.
671 DisambiguateGlobalSymbols(BD
.getProgram());
673 // Do the reduction...
674 ReduceMiscompilingFunctions(BD
, TestFn
).reduceList(MiscompiledFunctions
,
677 return MiscompiledFunctions
;
679 outs() << "\n*** The following function"
680 << (MiscompiledFunctions
.size() == 1 ? " is" : "s are")
681 << " being miscompiled: ";
682 PrintFunctionList(MiscompiledFunctions
);
687 return MiscompiledFunctions
;
690 /// TestOptimizer - This is the predicate function used to check to see if the
691 /// "Test" portion of the program is misoptimized. If so, return true. In any
692 /// case, both module arguments are deleted.
694 static bool TestOptimizer(BugDriver
&BD
, Module
*Test
, Module
*Safe
,
695 std::string
&Error
) {
696 // Run the optimization passes on ToOptimize, producing a transformed version
697 // of the functions being tested.
698 outs() << " Optimizing functions being tested: ";
699 Module
*Optimized
= BD
.runPassesOn(Test
, BD
.getPassesToRun(),
700 /*AutoDebugCrashes*/true);
704 outs() << " Checking to see if the merged program executes correctly: ";
706 Module
*New
= TestMergedProgram(BD
, Optimized
, Safe
, true, Error
, Broken
);
708 outs() << (Broken
? " nope.\n" : " yup.\n");
709 // Delete the original and set the new program.
710 delete BD
.swapProgramIn(New
);
716 /// debugMiscompilation - This method is used when the passes selected are not
717 /// crashing, but the generated output is semantically different from the
720 void BugDriver::debugMiscompilation(std::string
*Error
) {
721 // Make sure something was miscompiled...
722 if (!BugpointIsInterrupted
)
723 if (!ReduceMiscompilingPasses(*this).reduceList(PassesToRun
, *Error
)) {
725 errs() << "*** Optimized program matches reference output! No problem"
726 << " detected...\nbugpoint can't help you with your problem!\n";
730 outs() << "\n*** Found miscompiling pass"
731 << (getPassesToRun().size() == 1 ? "" : "es") << ": "
732 << getPassesString(getPassesToRun()) << '\n';
733 EmitProgressBitcode(Program
, "passinput");
735 std::vector
<Function
*> MiscompiledFunctions
=
736 DebugAMiscompilation(*this, TestOptimizer
, *Error
);
740 // Output a bunch of bitcode files for the user...
741 outs() << "Outputting reduced bitcode files which expose the problem:\n";
742 ValueToValueMapTy VMap
;
743 Module
*ToNotOptimize
= CloneModule(getProgram(), VMap
);
744 Module
*ToOptimize
= SplitFunctionsOutOfModule(ToNotOptimize
,
745 MiscompiledFunctions
,
748 outs() << " Non-optimized portion: ";
749 EmitProgressBitcode(ToNotOptimize
, "tonotoptimize", true);
750 delete ToNotOptimize
; // Delete hacked module.
752 outs() << " Portion that is input to optimizer: ";
753 EmitProgressBitcode(ToOptimize
, "tooptimize");
754 delete ToOptimize
; // Delete hacked module.
759 /// CleanupAndPrepareModules - Get the specified modules ready for code
760 /// generator testing.
762 static void CleanupAndPrepareModules(BugDriver
&BD
, Module
*&Test
,
764 // Clean up the modules, removing extra cruft that we don't need anymore...
765 Test
= BD
.performFinalCleanups(Test
);
767 // If we are executing the JIT, we have several nasty issues to take care of.
768 if (!BD
.isExecutingJIT()) return;
770 // First, if the main function is in the Safe module, we must add a stub to
771 // the Test module to call into it. Thus, we create a new function `main'
772 // which just calls the old one.
773 if (Function
*oldMain
= Safe
->getFunction("main"))
774 if (!oldMain
->isDeclaration()) {
776 oldMain
->setName("llvm_bugpoint_old_main");
777 // Create a NEW `main' function with same type in the test module.
778 Function
*newMain
= Function::Create(oldMain
->getFunctionType(),
779 GlobalValue::ExternalLinkage
,
781 // Create an `oldmain' prototype in the test module, which will
782 // corresponds to the real main function in the same module.
783 Function
*oldMainProto
= Function::Create(oldMain
->getFunctionType(),
784 GlobalValue::ExternalLinkage
,
785 oldMain
->getName(), Test
);
786 // Set up and remember the argument list for the main function.
787 std::vector
<Value
*> args
;
788 for (Function::arg_iterator
789 I
= newMain
->arg_begin(), E
= newMain
->arg_end(),
790 OI
= oldMain
->arg_begin(); I
!= E
; ++I
, ++OI
) {
791 I
->setName(OI
->getName()); // Copy argument names from oldMain
795 // Call the old main function and return its result
796 BasicBlock
*BB
= BasicBlock::Create(Safe
->getContext(), "entry", newMain
);
797 CallInst
*call
= CallInst::Create(oldMainProto
, args
.begin(), args
.end(),
800 // If the type of old function wasn't void, return value of call
801 ReturnInst::Create(Safe
->getContext(), call
, BB
);
804 // The second nasty issue we must deal with in the JIT is that the Safe
805 // module cannot directly reference any functions defined in the test
806 // module. Instead, we use a JIT API call to dynamically resolve the
809 // Add the resolver to the Safe module.
810 // Prototype: void *getPointerToNamedFunction(const char* Name)
811 Constant
*resolverFunc
=
812 Safe
->getOrInsertFunction("getPointerToNamedFunction",
813 Type::getInt8PtrTy(Safe
->getContext()),
814 Type::getInt8PtrTy(Safe
->getContext()),
817 // Use the function we just added to get addresses of functions we need.
818 for (Module::iterator F
= Safe
->begin(), E
= Safe
->end(); F
!= E
; ++F
) {
819 if (F
->isDeclaration() && !F
->use_empty() && &*F
!= resolverFunc
&&
820 !F
->isIntrinsic() /* ignore intrinsics */) {
821 Function
*TestFn
= Test
->getFunction(F
->getName());
823 // Don't forward functions which are external in the test module too.
824 if (TestFn
&& !TestFn
->isDeclaration()) {
825 // 1. Add a string constant with its name to the global file
826 Constant
*InitArray
= ConstantArray::get(F
->getContext(), F
->getName());
827 GlobalVariable
*funcName
=
828 new GlobalVariable(*Safe
, InitArray
->getType(), true /*isConstant*/,
829 GlobalValue::InternalLinkage
, InitArray
,
830 F
->getName() + "_name");
832 // 2. Use `GetElementPtr *funcName, 0, 0' to convert the string to an
833 // sbyte* so it matches the signature of the resolver function.
835 // GetElementPtr *funcName, ulong 0, ulong 0
836 std::vector
<Constant
*> GEPargs(2,
837 Constant::getNullValue(Type::getInt32Ty(F
->getContext())));
839 ConstantExpr::getGetElementPtr(funcName
, &GEPargs
[0], 2);
840 std::vector
<Value
*> ResolverArgs
;
841 ResolverArgs
.push_back(GEP
);
843 // Rewrite uses of F in global initializers, etc. to uses of a wrapper
844 // function that dynamically resolves the calls to F via our JIT API
845 if (!F
->use_empty()) {
846 // Create a new global to hold the cached function pointer.
847 Constant
*NullPtr
= ConstantPointerNull::get(F
->getType());
848 GlobalVariable
*Cache
=
849 new GlobalVariable(*F
->getParent(), F
->getType(),
850 false, GlobalValue::InternalLinkage
,
851 NullPtr
,F
->getName()+".fpcache");
853 // Construct a new stub function that will re-route calls to F
854 const FunctionType
*FuncTy
= F
->getFunctionType();
855 Function
*FuncWrapper
= Function::Create(FuncTy
,
856 GlobalValue::InternalLinkage
,
857 F
->getName() + "_wrapper",
859 BasicBlock
*EntryBB
= BasicBlock::Create(F
->getContext(),
860 "entry", FuncWrapper
);
861 BasicBlock
*DoCallBB
= BasicBlock::Create(F
->getContext(),
862 "usecache", FuncWrapper
);
863 BasicBlock
*LookupBB
= BasicBlock::Create(F
->getContext(),
864 "lookupfp", FuncWrapper
);
866 // Check to see if we already looked up the value.
867 Value
*CachedVal
= new LoadInst(Cache
, "fpcache", EntryBB
);
868 Value
*IsNull
= new ICmpInst(*EntryBB
, ICmpInst::ICMP_EQ
, CachedVal
,
870 BranchInst::Create(LookupBB
, DoCallBB
, IsNull
, EntryBB
);
872 // Resolve the call to function F via the JIT API:
874 // call resolver(GetElementPtr...)
876 CallInst::Create(resolverFunc
, ResolverArgs
.begin(),
877 ResolverArgs
.end(), "resolver", LookupBB
);
879 // Cast the result from the resolver to correctly-typed function.
880 CastInst
*CastedResolver
=
881 new BitCastInst(Resolver
,
882 PointerType::getUnqual(F
->getFunctionType()),
883 "resolverCast", LookupBB
);
885 // Save the value in our cache.
886 new StoreInst(CastedResolver
, Cache
, LookupBB
);
887 BranchInst::Create(DoCallBB
, LookupBB
);
889 PHINode
*FuncPtr
= PHINode::Create(NullPtr
->getType(), 2,
891 FuncPtr
->addIncoming(CastedResolver
, LookupBB
);
892 FuncPtr
->addIncoming(CachedVal
, EntryBB
);
894 // Save the argument list.
895 std::vector
<Value
*> Args
;
896 for (Function::arg_iterator i
= FuncWrapper
->arg_begin(),
897 e
= FuncWrapper
->arg_end(); i
!= e
; ++i
)
900 // Pass on the arguments to the real function, return its result
901 if (F
->getReturnType()->isVoidTy()) {
902 CallInst::Create(FuncPtr
, Args
.begin(), Args
.end(), "", DoCallBB
);
903 ReturnInst::Create(F
->getContext(), DoCallBB
);
905 CallInst
*Call
= CallInst::Create(FuncPtr
, Args
.begin(), Args
.end(),
907 ReturnInst::Create(F
->getContext(),Call
, DoCallBB
);
910 // Use the wrapper function instead of the old function
911 F
->replaceAllUsesWith(FuncWrapper
);
917 if (verifyModule(*Test
) || verifyModule(*Safe
)) {
918 errs() << "Bugpoint has a bug, which corrupted a module!!\n";
925 /// TestCodeGenerator - This is the predicate function used to check to see if
926 /// the "Test" portion of the program is miscompiled by the code generator under
927 /// test. If so, return true. In any case, both module arguments are deleted.
929 static bool TestCodeGenerator(BugDriver
&BD
, Module
*Test
, Module
*Safe
,
930 std::string
&Error
) {
931 CleanupAndPrepareModules(BD
, Test
, Safe
);
933 sys::Path
TestModuleBC("bugpoint.test.bc");
935 if (TestModuleBC
.makeUnique(true, &ErrMsg
)) {
936 errs() << BD
.getToolName() << "Error making unique filename: "
940 if (BD
.writeProgramToFile(TestModuleBC
.str(), Test
)) {
941 errs() << "Error writing bitcode to `" << TestModuleBC
.str()
947 FileRemover
TestModuleBCRemover(TestModuleBC
.str(), !SaveTemps
);
949 // Make the shared library
950 sys::Path
SafeModuleBC("bugpoint.safe.bc");
951 if (SafeModuleBC
.makeUnique(true, &ErrMsg
)) {
952 errs() << BD
.getToolName() << "Error making unique filename: "
957 if (BD
.writeProgramToFile(SafeModuleBC
.str(), Safe
)) {
958 errs() << "Error writing bitcode to `" << SafeModuleBC
.str()
963 FileRemover
SafeModuleBCRemover(SafeModuleBC
.str(), !SaveTemps
);
965 std::string SharedObject
= BD
.compileSharedObject(SafeModuleBC
.str(), Error
);
970 FileRemover
SharedObjectRemover(SharedObject
, !SaveTemps
);
972 // Run the code generator on the `Test' code, loading the shared library.
973 // The function returns whether or not the new output differs from reference.
974 bool Result
= BD
.diffProgram(BD
.getProgram(), TestModuleBC
.str(),
975 SharedObject
, false, &Error
);
980 errs() << ": still failing!\n";
982 errs() << ": didn't fail.\n";
988 /// debugCodeGenerator - debug errors in LLC, LLI, or CBE.
990 bool BugDriver::debugCodeGenerator(std::string
*Error
) {
991 if ((void*)SafeInterpreter
== (void*)Interpreter
) {
992 std::string Result
= executeProgramSafely(Program
, "bugpoint.safe.out",
994 if (Error
->empty()) {
995 outs() << "\n*** The \"safe\" i.e. 'known good' backend cannot match "
996 << "the reference diff. This may be due to a\n front-end "
997 << "bug or a bug in the original program, but this can also "
998 << "happen if bugpoint isn't running the program with the "
999 << "right flags or input.\n I left the result of executing "
1000 << "the program with the \"safe\" backend in this file for "
1002 << Result
<< "'.\n";
1007 DisambiguateGlobalSymbols(Program
);
1009 std::vector
<Function
*> Funcs
= DebugAMiscompilation(*this, TestCodeGenerator
,
1011 if (!Error
->empty())
1014 // Split the module into the two halves of the program we want.
1015 ValueToValueMapTy VMap
;
1016 Module
*ToNotCodeGen
= CloneModule(getProgram(), VMap
);
1017 Module
*ToCodeGen
= SplitFunctionsOutOfModule(ToNotCodeGen
, Funcs
, VMap
);
1019 // Condition the modules
1020 CleanupAndPrepareModules(*this, ToCodeGen
, ToNotCodeGen
);
1022 sys::Path
TestModuleBC("bugpoint.test.bc");
1024 if (TestModuleBC
.makeUnique(true, &ErrMsg
)) {
1025 errs() << getToolName() << "Error making unique filename: "
1030 if (writeProgramToFile(TestModuleBC
.str(), ToCodeGen
)) {
1031 errs() << "Error writing bitcode to `" << TestModuleBC
.str()
1037 // Make the shared library
1038 sys::Path
SafeModuleBC("bugpoint.safe.bc");
1039 if (SafeModuleBC
.makeUnique(true, &ErrMsg
)) {
1040 errs() << getToolName() << "Error making unique filename: "
1045 if (writeProgramToFile(SafeModuleBC
.str(), ToNotCodeGen
)) {
1046 errs() << "Error writing bitcode to `" << SafeModuleBC
.str()
1050 std::string SharedObject
= compileSharedObject(SafeModuleBC
.str(), *Error
);
1051 if (!Error
->empty())
1053 delete ToNotCodeGen
;
1055 outs() << "You can reproduce the problem with the command line: \n";
1056 if (isExecutingJIT()) {
1057 outs() << " lli -load " << SharedObject
<< " " << TestModuleBC
.str();
1059 outs() << " llc " << TestModuleBC
.str() << " -o " << TestModuleBC
.str()
1061 outs() << " gcc " << SharedObject
<< " " << TestModuleBC
.str()
1062 << ".s -o " << TestModuleBC
.str() << ".exe";
1063 #if defined (HAVE_LINK_R)
1064 outs() << " -Wl,-R.";
1067 outs() << " " << TestModuleBC
.str() << ".exe";
1069 for (unsigned i
= 0, e
= InputArgv
.size(); i
!= e
; ++i
)
1070 outs() << " " << InputArgv
[i
];
1072 outs() << "The shared object was created with:\n llc -march=c "
1073 << SafeModuleBC
.str() << " -o temporary.c\n"
1074 << " gcc -xc temporary.c -O2 -o " << SharedObject
;
1075 if (TargetTriple
.getArch() == Triple::sparc
)
1076 outs() << " -G"; // Compile a shared library, `-G' for Sparc
1078 outs() << " -fPIC -shared"; // `-shared' for Linux/X86, maybe others
1080 outs() << " -fno-strict-aliasing\n";