[AMDGPU] Don’t marke the .note section as ALLOC
authorSebastian Neubauer <sebastian.neubauer@amd.com>
Thu, 13 Feb 2020 08:43:08 +0000 (09:43 +0100)
committerSebastian Neubauer <sebastian.neubauer@amd.com>
Thu, 20 Feb 2020 14:14:48 +0000 (15:14 +0100)
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

llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.cpp
llvm/test/CodeGen/AMDGPU/hsa.ll

index c7ccd0a..3c52de9 100644 (file)
@@ -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
index 3462398..c78239e 100644 (file)
@@ -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