AMDGPU/SI: Emit global variable sizes when targeting HSA

[arsenm]

LGTM

lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp

156

I think you should move this cast up to the getSymbol

lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp

void AMDGPUAsmPrinter::EmitGlobalVariable(const GlobalVariable *GV) {
   AMDGPUTargetStreamer *TS =
       static_cast<AMDGPUTargetStreamer *>(OutStreamer->getTargetStreamer());
   if (isModuleLinkage(GV)) {
     TS->EmitAMDGPUHsaModuleScopeGlobal(GV->getName());
   } else {
     TS->EmitAMDGPUHsaProgramScopeGlobal(GV->getName());
   }
 
+  MCSymbol *GVSym = getSymbol(GV);
   const DataLayout &DL = getDataLayout();
+
+  // Emit the size
+  uint64_t Size = DL.getTypeAllocSize(GV->getType()->getElementType());
+  OutStreamer->emitELFSize(cast<MCSymbolELF>(GVSym),
+                           MCConstantExpr::create(Size, OutContext));
   OutStreamer->PushSection();
   OutStreamer->SwitchSection(
       getObjFileLowering().SectionForGlobal(GV, *Mang, TM));
-  MCSymbol *GVSym = getSymbol(GV);
   const Constant *C = GV->getInitializer();
   OutStreamer->EmitLabel(GVSym);
   EmitGlobalConstant(DL, C);
   OutStreamer->PopSection();
 }
 
 bool AMDGPUAsmPrinter::runOnMachineFunction(MachineFunction &MF) {
 

test/CodeGen/AMDGPU/hsa-globals.ll

 
 @internal_readonly = internal unnamed_addr addrspace(2) constant i32 0
 @external_readonly = unnamed_addr addrspace(2) constant i32 0
 
 define void @test() {
   ret void
 }
 
+; ASM: .size internal_global_program, 4
 ; ASM: .amdgpu_hsa_module_global internal_global
 ; ASM: .hsadata_global_program
 ; ASM: internal_global_program:
 ; ASM: .long 0
 
+; ASM: .size common_global_program, 4
 ; ASM: .amdgpu_hsa_module_global common_global
 ; ASM: .hsadata_global_program
 ; ASM: common_global_program:
 ; ASM: .long 0
 
+; ASM: .size external_global_program, 4
 ; ASM: .amdgpu_hsa_program_global external_global
 ; ASM: .hsadata_global_program
 ; ASM: external_global_program:
 ; ASM: .long 0
 
+; ASM: .size internal_global_agent, 4
 ; ASM: .amdgpu_hsa_module_global internal_global
 ; ASM: .hsadata_global_agent
 ; ASM: internal_global_agent:
 ; ASM: .long 0
 
+; ASM: .size common_global_agent, 4
 ; ASM: .amdgpu_hsa_module_global common_global
 ; ASM: .hsadata_global_agent
 ; ASM: common_global_agent:
 ; ASM: .long 0
 
+; ASM: .size external_global_agent, 4
 ; ASM: .amdgpu_hsa_program_global external_global
 ; ASM: .hsadata_global_agent
 ; ASM: external_global_agent:
 ; ASM: .long 0
 
+; ASM: .size internal_readonly, 4
 ; ASM: .amdgpu_hsa_module_global internal_readonly
 ; ASM: .hsatext
 ; ASM: internal_readonly:
 ; ASM: .long 0
 
+; ASM: .size external_readonly, 4
 ; ASM: .amdgpu_hsa_program_global external_readonly
 ; ASM: .hsatext
 ; ASM: external_readonly:
 ; ASM: .long 0
 
 ; ELF: Section {
 ; ELF: Name: .hsadata_global_program
 ; ELF: Type: SHT_PROGBITS (0x1)

 ; ELF: SHF_AMDGPU_HSA_AGENT (0x800000)
 ; ELF: SHF_AMDGPU_HSA_GLOBAL (0x100000)
 ; ELF: SHF_WRITE (0x1)
 ; ELF: ]
 ; ELF: }
 
 ; ELF: Symbol {
 ; ELF: Name: common_global_agent
+; ELF: Size: 4
 ; ELF: Binding: Local
 ; ELF: Section: .hsadata_global_agent
 ; ELF: }
 
 ; ELF: Symbol {
 ; ELF: Name: common_global_program
+; ELF: Size: 4
 ; ELF: Binding: Local
 ; ELF: Section: .hsadata_global_program
 ; ELF: }
 
 ; ELF: Symbol {
 ; ELF: Name: internal_global_agent
+; ELF: Size: 4
 ; ELF: Binding: Local
 ; ELF: Type: Object
 ; ELF: Section: .hsadata_global_agent
 ; ELF: }
 
 ; ELF: Symbol {
 ; ELF: Name: internal_global_program
+; ELF: Size: 4
 ; ELF: Binding: Local
 ; ELF: Type: Object
 ; ELF: Section: .hsadata_global_program
 ; ELF: }
 
 ; ELF: Symbol {
 ; ELF: Name: internal_readonly
+; ELF: Size: 4
 ; ELF: Binding: Local
 ; ELF: Type: Object
 ; ELF: Section: .hsatext
 ; ELF: }
 
 ; ELF: Symbol {
 ; ELF: Name: external_global_agent
+; ELF: Size: 4
 ; ELF: Binding: Global
 ; ELF: Type: Object
 ; ELF: Section: .hsadata_global_agent
 ; ELF: }
 
 ; ELF: Symbol {
 ; ELF: Name: external_global_program
+; ELF: Size: 4
 ; ELF: Binding: Global
 ; ELF: Type: Object
 ; ELF: Section: .hsadata_global_program
 ; ELF: }
 
 ; ELF: Symbol {
 ; ELF: Name: external_readonly
+; ELF: Size: 4
 ; ELF: Binding: Global
 ; ELF: Type: Object
 ; ELF: Section: .hsatext
 ; ELF: }