diff options
| author | Jay Foad <jay.foad@gmail.com> | 2019-09-02 14:40:57 +0000 |
|---|---|---|
| committer | Jay Foad <jay.foad@gmail.com> | 2019-09-02 14:40:57 +0000 |
| commit | 6e18266aa4dd78953557b8614cb9ff260bad7c65 (patch) | |
| tree | 41f6b1039acd58a35d0308405f4a67f7df7d9c6d /llvm/lib/Target/AMDGPU/Utils | |
| parent | 4aa90ea58efbd09de834b2939fee91a47059200d (diff) | |
| download | bcm5719-llvm-6e18266aa4dd78953557b8614cb9ff260bad7c65.tar.gz bcm5719-llvm-6e18266aa4dd78953557b8614cb9ff260bad7c65.zip | |
Partially revert D61491 "AMDGPU: Be explicit about whether the high-word in SI_PC_ADD_REL_OFFSET is 0"
Summary:
D61491 caused us to use relocs when they're not strictly necessary, to
refer to symbols in the text section. This is a pessimization and it's a
problem for some loaders that don't support relocs yet.
Reviewers: nhaehnle, arsenm, tpr
Subscribers: kzhuravl, jvesely, wdng, yaxunl, dstuttard, t-tye, hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D65813
llvm-svn: 370667
Diffstat (limited to 'llvm/lib/Target/AMDGPU/Utils')
| -rw-r--r-- | llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp b/llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp index 22a642616b4..bb4169788f4 100644 --- a/llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp +++ b/llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp @@ -514,7 +514,7 @@ bool isReadOnlySegment(const GlobalValue *GV) { } bool shouldEmitConstantsToTextSection(const Triple &TT) { - return TT.getOS() != Triple::AMDHSA; + return TT.getOS() == Triple::AMDPAL; } int getIntegerAttribute(const Function &F, StringRef Name, int Default) { |
