diff options
| author | Marek Olsak <marek.olsak@amd.com> | 2018-04-10 22:48:23 +0000 |
|---|---|---|
| committer | Marek Olsak <marek.olsak@amd.com> | 2018-04-10 22:48:23 +0000 |
| commit | a9a58fa236ab19b5caae32330d31e30ebdf6751f (patch) | |
| tree | 907bcacd4096bc2131352fe96717b0d2a783929b /llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp | |
| parent | 3dc27f1a6914b1fb7dcbc152dc71ea6092f125a8 (diff) | |
| download | bcm5719-llvm-a9a58fa236ab19b5caae32330d31e30ebdf6751f.tar.gz bcm5719-llvm-a9a58fa236ab19b5caae32330d31e30ebdf6751f.zip | |
AMDGPU: enable 128-bit for local addr space under an option
Author: Samuel Pitoiset
ds_read_b128 and ds_write_b128 have been recently enabled
under the amdgpu-ds128 option because the performance benefit
is unclear.
Though, using 128-bit loads/stores for the local address space
appears to introduce regressions in tessellation shaders. Not
sure what is broken, but as ds_read_b128/ds_write_b128 are not
enabled by default, just introduce a global option and enable
128-bit only if requested (until it's fixed/used correctly).
v2: - fix regressions in merge-stores.ll and multiple_tails.ll
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105464
llvm-svn: 329764
Diffstat (limited to 'llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp')
| -rw-r--r-- | llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp b/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp index 27b5799245b..07ed04e41d7 100644 --- a/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp +++ b/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp @@ -132,6 +132,7 @@ AMDGPUSubtarget::AMDGPUSubtarget(const Triple &TT, StringRef GPU, StringRef FS, EnableLoadStoreOpt(false), EnableUnsafeDSOffsetFolding(false), EnableSIScheduler(false), + EnableDS128(false), DumpCode(false), FP64(false), |
