diff options
| author | Manman Ren <mren@apple.com> | 2013-05-14 21:52:44 +0000 |
|---|---|---|
| committer | Manman Ren <mren@apple.com> | 2013-05-14 21:52:44 +0000 |
| commit | 5150270d071f05c6ea1fd26ef448954ddb5746dc (patch) | |
| tree | 2e628021377421f2d8b9834f4a579363766736ca /lib/Transforms | |
| parent | e83a6f32a408d16f1fa07c658236eb226ea332fb (diff) | |
| download | llvm-5150270d071f05c6ea1fd26ef448954ddb5746dc.tar.gz llvm-5150270d071f05c6ea1fd26ef448954ddb5746dc.tar.bz2 llvm-5150270d071f05c6ea1fd26ef448954ddb5746dc.tar.xz | |
GlobalOpt: fix an issue where CXAAtExitFn points to a deleted function.
CXAAtExitFn was set outside a loop and before optimizations where functions
can be deleted. This patch will set CXAAtExitFn inside the loop and after
optimizations.
Seg fault when running LTO because of accesses to a deleted function.
rdar://problem/13838828
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@181838 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/IPO/GlobalOpt.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/Transforms/IPO/GlobalOpt.cpp b/lib/Transforms/IPO/GlobalOpt.cpp index 0ef900e2b9..4a9cb27b03 100644 --- a/lib/Transforms/IPO/GlobalOpt.cpp +++ b/lib/Transforms/IPO/GlobalOpt.cpp @@ -3323,8 +3323,6 @@ bool GlobalOpt::runOnModule(Module &M) { // Try to find the llvm.globalctors list. GlobalVariable *GlobalCtors = FindGlobalCtors(M); - Function *CXAAtExitFn = FindCXAAtExit(M, TLI); - bool LocalChange = true; while (LocalChange) { LocalChange = false; @@ -3342,7 +3340,9 @@ bool GlobalOpt::runOnModule(Module &M) { // Resolve aliases, when possible. LocalChange |= OptimizeGlobalAliases(M); - // Try to remove trivial global destructors. + // Try to remove trivial global destructors if they are not removed + // already. + Function *CXAAtExitFn = FindCXAAtExit(M, TLI); if (CXAAtExitFn) LocalChange |= OptimizeEmptyGlobalCXXDtors(CXAAtExitFn); |