Skip to content

Commit fb0b5f5

Browse files
committed
AMDGPU/NFC: Move AMDGPU specific note types to ELF.h
Differential Revision: https://reviews.llvm.org/D38747 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@315608 91177308-0d34-0410-b5e6-96231b3b80d8
1 parent 6beab2e commit fb0b5f5

File tree

4 files changed

+14
-10
lines changed

4 files changed

+14
-10
lines changed

include/llvm/BinaryFormat/ELF.h

+8
Original file line numberDiff line numberDiff line change
@@ -1369,6 +1369,14 @@ enum {
13691369
NT_GNU_GOLD_VERSION = 4,
13701370
};
13711371

1372+
// AMDGPU specific notes.
1373+
enum {
1374+
// Note types with values between 0 and 9 (inclusive) are reserved.
1375+
NT_AMD_AMDGPU_HSA_METADATA = 10,
1376+
NT_AMD_AMDGPU_ISA = 11,
1377+
NT_AMD_AMDGPU_PAL_METADATA = 12
1378+
};
1379+
13721380
enum {
13731381
GNU_ABI_TAG_LINUX = 0,
13741382
GNU_ABI_TAG_HURD = 1,

lib/Target/AMDGPU/AMDGPUPTNote.h

+1-4
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ const char SectionName[] = ".note";
2525

2626
const char NoteName[] = "AMD";
2727

28-
// TODO: Move this enum to include/llvm/Support so it can be used in tools?
28+
// TODO: Remove this file once we drop code object v2.
2929
enum NoteType{
3030
NT_AMDGPU_HSA_RESERVED_0 = 0,
3131
NT_AMDGPU_HSA_CODE_OBJECT_VERSION = 1,
@@ -37,9 +37,6 @@ enum NoteType{
3737
NT_AMDGPU_HSA_RESERVED_7 = 7,
3838
NT_AMDGPU_HSA_RESERVED_8 = 8,
3939
NT_AMDGPU_HSA_RESERVED_9 = 9,
40-
NT_AMDGPU_HSA_CODE_OBJECT_METADATA = 10,
41-
NT_AMD_AMDGPU_ISA = 11,
42-
NT_AMDGPU_PAL_METADATA = 12,
4340
NT_AMDGPU_HSA_HLDEBUG_DEBUG = 101,
4441
NT_AMDGPU_HSA_HLDEBUG_TARGET = 102
4542
};

lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.cpp

+4-4
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ MCELFStreamer &AMDGPUTargetELFStreamer::getStreamer() {
128128
}
129129

130130
void AMDGPUTargetELFStreamer::EmitAMDGPUNote(
131-
const MCExpr *DescSZ, ElfNote::NoteType Type,
131+
const MCExpr *DescSZ, unsigned NoteType,
132132
function_ref<void(MCELFStreamer &)> EmitDesc) {
133133
auto &S = getStreamer();
134134
auto &Context = S.getContext();
@@ -140,7 +140,7 @@ void AMDGPUTargetELFStreamer::EmitAMDGPUNote(
140140
ElfNote::SectionName, ELF::SHT_NOTE, ELF::SHF_ALLOC));
141141
S.EmitIntValue(NameSZ, 4); // namesz
142142
S.EmitValue(DescSZ, 4); // descz
143-
S.EmitIntValue(Type, 4); // type
143+
S.EmitIntValue(NoteType, 4); // type
144144
S.EmitBytes(StringRef(ElfNote::NoteName, NameSZ)); // name
145145
S.EmitValueToAlignment(4, 0, 1, 0); // padding 0
146146
EmitDesc(S); // desc
@@ -225,7 +225,7 @@ bool AMDGPUTargetELFStreamer::EmitHSAMetadata(
225225

226226
EmitAMDGPUNote(
227227
DescSZ,
228-
ElfNote::NT_AMDGPU_HSA_CODE_OBJECT_METADATA,
228+
ELF::NT_AMD_AMDGPU_HSA_METADATA,
229229
[&](MCELFStreamer &OS) {
230230
OS.EmitLabel(DescBegin);
231231
OS.EmitBytes(HSAMetadataString);
@@ -239,7 +239,7 @@ bool AMDGPUTargetELFStreamer::EmitPALMetadata(
239239
const PALMD::Metadata &PALMetadata) {
240240
EmitAMDGPUNote(
241241
MCConstantExpr::create(PALMetadata.size() * sizeof(uint32_t), getContext()),
242-
ElfNote::NT_AMDGPU_PAL_METADATA,
242+
ELF::NT_AMD_AMDGPU_PAL_METADATA,
243243
[&](MCELFStreamer &OS){
244244
for (auto I : PALMetadata)
245245
OS.EmitIntValue(I, sizeof(uint32_t));

lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.h

+1-2
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,7 @@ class AMDGPUTargetAsmStreamer final : public AMDGPUTargetStreamer {
7878
class AMDGPUTargetELFStreamer final : public AMDGPUTargetStreamer {
7979
MCStreamer &Streamer;
8080

81-
void EmitAMDGPUNote(const MCExpr *DescSize,
82-
AMDGPU::ElfNote::NoteType Type,
81+
void EmitAMDGPUNote(const MCExpr *DescSize, unsigned NoteType,
8382
function_ref<void(MCELFStreamer &)> EmitDesc);
8483

8584
public:

0 commit comments

Comments
 (0)