diff options
| author | David Majnemer <david.majnemer@gmail.com> | 2014-06-26 03:02:19 +0000 |
|---|---|---|
| committer | David Majnemer <david.majnemer@gmail.com> | 2014-06-26 03:02:19 +0000 |
| commit | 6098b2f5199c90ac36fda3cf6f049b72aacee27d (patch) | |
| tree | f2fe726dbc3f2ffe4573887653a84539b90e2000 /llvm/lib/Transforms | |
| parent | 0843bff137fa43719744fba9d41fad4c827ab458 (diff) | |
| download | bcm5719-llvm-6098b2f5199c90ac36fda3cf6f049b72aacee27d.tar.gz bcm5719-llvm-6098b2f5199c90ac36fda3cf6f049b72aacee27d.zip | |
GlobalOpt: Don't optimize thread_local for initializers
Folding a reference to a thread_local variable into another global
variable's initializer is very problematic, there is no relocation that
exists to represent such an access.
llvm-svn: 211762
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/IPO/GlobalOpt.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/IPO/GlobalOpt.cpp b/llvm/lib/Transforms/IPO/GlobalOpt.cpp index 75c1878f3fd..36c04f092bc 100644 --- a/llvm/lib/Transforms/IPO/GlobalOpt.cpp +++ b/llvm/lib/Transforms/IPO/GlobalOpt.cpp @@ -1980,9 +1980,10 @@ isSimpleEnoughValueToCommit(Constant *C, static bool isSimpleEnoughValueToCommitHelper(Constant *C, SmallPtrSet<Constant*, 8> &SimpleConstants, const DataLayout *DL) { - // Simple global addresses are supported, do not allow dllimport globals. + // Simple global addresses are supported, do not allow dllimport or + // thread-local globals. if (auto *GV = dyn_cast<GlobalValue>(C)) - return !GV->hasDLLImportStorageClass(); + return !GV->hasDLLImportStorageClass() && !GV->isThreadLocal(); // Simple integer, undef, constant aggregate zero, etc are all supported. if (C->getNumOperands() == 0 || isa<BlockAddress>(C)) |
