diff options
| author | Daniel Dunbar <daniel@zuster.org> | 2009-06-03 21:51:32 +0000 |
|---|---|---|
| committer | Daniel Dunbar <daniel@zuster.org> | 2009-06-03 21:51:32 +0000 |
| commit | e0f0e0be1c5f6f80cd866d84643509ef399eec96 (patch) | |
| tree | 7cddfb427529cb31be6167b88a76846e707d0651 /tools/llvm-ld | |
| parent | a8ba2c25e9ea7b0d213b485debe5d044efde66a4 (diff) | |
| download | external_llvm-e0f0e0be1c5f6f80cd866d84643509ef399eec96.zip external_llvm-e0f0e0be1c5f6f80cd866d84643509ef399eec96.tar.gz external_llvm-e0f0e0be1c5f6f80cd866d84643509ef399eec96.tar.bz2 | |
Change LTO to run the global opt pass twice.
- This matches llvm-ld.
It took a bit of archeology to figure out what the right thing to do was
(whether this was intentionally added or intentionally removed). My final
conclusion is that Chris added this intentionally here:
http://llvm.org/viewvc/llvm-project?view=rev&revision=16913
but the changes weren't propogated to llvm-ld until here:
http://llvm.org/viewvc/llvm-project?view=rev&revision=34058
which was after lto.cpp had been cloned off (of llvm-ld), here:
http://llvm.org/viewvc/llvm-project?view=rev&revision=29494
From the commit message, it looks like the motivation for running global opt
again is because we ran it prior to inlining. Based on that I updated the
comment and also only run the pass if we actually ran the inliner.
Chris, please review.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@72811 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'tools/llvm-ld')
| -rw-r--r-- | tools/llvm-ld/Optimize.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/llvm-ld/Optimize.cpp b/tools/llvm-ld/Optimize.cpp index a4ca951..e466895 100644 --- a/tools/llvm-ld/Optimize.cpp +++ b/tools/llvm-ld/Optimize.cpp @@ -94,7 +94,7 @@ void Optimize(Module* M) { if (!DisableOptimizations) createStandardLTOPasses(&Passes, !DisableInternalize, !DisableInline, - /*RunSecondGlobalOpt=*/true, VerifyEach); + VerifyEach); // If the -s or -S command line options were specified, strip the symbols out // of the resulting program to make it smaller. -s and -S are GNU ld options |