diff options
author | Chandler Carruth <chandlerc@gmail.com> | 2012-04-08 17:51:45 +0000 |
---|---|---|
committer | Chandler Carruth <chandlerc@gmail.com> | 2012-04-08 17:51:45 +0000 |
commit | 253933ee9ef2c413ecd782efeacc5d7b9bcda09a (patch) | |
tree | 4dd402f01a78ae40c9d6569bc5fba37f5af64501 /lib/Target/TargetMachine.cpp | |
parent | 34797136cb9fa9f450c0e1c47983482083979dd4 (diff) |
Teach LLVM about a PIE option which, when enabled on top of PIC, makes
optimizations which are valid for position independent code being linked
into a single executable, but not for such code being linked into
a shared library.
I discussed the design of this with Eric Christopher, and the decision
was to support an optional bit rather than a completely separate
relocation model. Fundamentally, this is still PIC relocation, its just
that certain optimizations are only valid under a PIC relocation model
when the resulting code won't be in a shared library. The simplest path
to here is to expose a single bit option in the TargetOptions. If folks
have different/better designs, I'm all ears. =]
I've included the first optimization based upon this: changing TLS
models to the *Exec models when PIE is enabled. This is the LLVM
component of PR12380 and is all of the hard work.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@154294 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/TargetMachine.cpp')
-rw-r--r-- | lib/Target/TargetMachine.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/Target/TargetMachine.cpp b/lib/Target/TargetMachine.cpp index b8e7f15d09..b9b2526876 100644 --- a/lib/Target/TargetMachine.cpp +++ b/lib/Target/TargetMachine.cpp @@ -82,7 +82,8 @@ TLSModel::Model TargetMachine::getTLSModel(const GlobalValue *GV) const { // For variables, is internal different from hidden? bool isHidden = GV->hasHiddenVisibility(); - if (getRelocationModel() == Reloc::PIC_) { + if (getRelocationModel() == Reloc::PIC_ && + !Options.PositionIndependentExecutable) { if (isLocal || isHidden) return TLSModel::LocalDynamic; else |