[llvm] 977cd66 - [AMDGPU] Don’t marke the .note section as ALLOC
Sebastian Neubauer via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 20 06:32:46 PST 2020
Author: Sebastian Neubauer
Date: 2020-02-20T15:14:48+01:00
New Revision: 977cd661cf019039dec7ffdd15bf0ac500828c87
URL: https://github.com/llvm/llvm-project/commit/977cd661cf019039dec7ffdd15bf0ac500828c87
DIFF: https://github.com/llvm/llvm-project/commit/977cd661cf019039dec7ffdd15bf0ac500828c87.diff
LOG: [AMDGPU] Don’t marke the .note section as ALLOC
Marking a section as ALLOC tells the ELF loader to load the section into memory.
As we do not want to load the notes into VRAM, the flag should not be there.
Differential Revision: https://reviews.llvm.org/D74600
Added:
Modified:
llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.cpp
llvm/test/CodeGen/AMDGPU/hsa.ll
Removed:
################################################################################
diff --git a/llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.cpp b/llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.cpp
index c7ccd0aa04ac..3c52de948c5c 100644
--- a/llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.cpp
+++ b/llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.cpp
@@ -440,7 +440,7 @@ void AMDGPUTargetELFStreamer::EmitNote(
S.PushSection();
S.SwitchSection(Context.getELFSection(
- ElfNote::SectionName, ELF::SHT_NOTE, ELF::SHF_ALLOC));
+ ElfNote::SectionName, ELF::SHT_NOTE, 0));
S.emitIntValue(NameSZ, 4); // namesz
S.emitValue(DescSZ, 4); // descz
S.emitIntValue(NoteType, 4); // type
diff --git a/llvm/test/CodeGen/AMDGPU/hsa.ll b/llvm/test/CodeGen/AMDGPU/hsa.ll
index 3462398b5851..c78239e2668a 100644
--- a/llvm/test/CodeGen/AMDGPU/hsa.ll
+++ b/llvm/test/CodeGen/AMDGPU/hsa.ll
@@ -19,8 +19,7 @@
; ELF: }
; ELF: SHT_NOTE
-; ELF: Flags [ (0x2)
-; ELF: SHF_ALLOC (0x2)
+; ELF: Flags [ (0x0)
; ELF: ]
; ELF: SectionData (
; ELF: 0000: 04000000 08000000 01000000 414D4400
More information about the llvm-commits
mailing list