Index: lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp =================================================================== --- lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp +++ lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp @@ -135,8 +135,9 @@ if (MFI->isKernel() && STM.isAmdCodeObjectV2()) { AMDGPUTargetStreamer *TS = static_cast(OutStreamer->getTargetStreamer()); - TS->EmitAMDGPUSymbolType(CurrentFnSym->getName(), - ELF::STT_AMDGPU_HSA_KERNEL); + SmallString<128> SymbolName; + getNameWithPrefix(SymbolName, MF->getFunction()), + TS->EmitAMDGPUSymbolType(SymbolName, ELF::STT_AMDGPU_HSA_KERNEL); } AsmPrinter::EmitFunctionEntryLabel(); Index: lib/Target/AMDGPU/AMDGPUMCInstLower.h =================================================================== --- lib/Target/AMDGPU/AMDGPUMCInstLower.h +++ lib/Target/AMDGPU/AMDGPUMCInstLower.h @@ -13,6 +13,7 @@ namespace llvm { class AMDGPUSubtarget; +class AsmPrinter; class MachineInstr; class MCContext; class MCInst; @@ -20,9 +21,11 @@ class AMDGPUMCInstLower { MCContext &Ctx; const AMDGPUSubtarget &ST; + const AsmPrinter &AP; public: - AMDGPUMCInstLower(MCContext &ctx, const AMDGPUSubtarget &ST); + AMDGPUMCInstLower(MCContext &ctx, const AMDGPUSubtarget &ST, + const AsmPrinter &AP); /// \brief Lower a MachineInstr to an MCInst void lower(const MachineInstr *MI, MCInst &OutMI) const; Index: lib/Target/AMDGPU/AMDGPUMCInstLower.cpp =================================================================== --- lib/Target/AMDGPU/AMDGPUMCInstLower.cpp +++ lib/Target/AMDGPU/AMDGPUMCInstLower.cpp @@ -36,8 +36,9 @@ using namespace llvm; -AMDGPUMCInstLower::AMDGPUMCInstLower(MCContext &ctx, const AMDGPUSubtarget &st): - Ctx(ctx), ST(st) { } +AMDGPUMCInstLower::AMDGPUMCInstLower(MCContext &ctx, const AMDGPUSubtarget &st, + const AsmPrinter &ap): + Ctx(ctx), ST(st), AP(ap) { } static MCSymbolRefExpr::VariantKind getVariantKind(unsigned MOFlags) { switch (MOFlags) { @@ -75,7 +76,9 @@ break; case MachineOperand::MO_GlobalAddress: { const GlobalValue *GV = MO.getGlobal(); - MCSymbol *Sym = Ctx.getOrCreateSymbol(StringRef(GV->getName())); + SmallString<128> SymbolName; + AP.getNameWithPrefix(SymbolName, GV); + MCSymbol *Sym = Ctx.getOrCreateSymbol(SymbolName); const MCExpr *SymExpr = MCSymbolRefExpr::create(Sym, getVariantKind(MO.getTargetFlags()),Ctx); const MCExpr *Expr = MCBinaryExpr::createAdd(SymExpr, @@ -97,7 +100,7 @@ void AMDGPUAsmPrinter::EmitInstruction(const MachineInstr *MI) { const AMDGPUSubtarget &STI = MF->getSubtarget(); - AMDGPUMCInstLower MCInstLowering(OutContext, STI); + AMDGPUMCInstLower MCInstLowering(OutContext, STI, *this); StringRef Err; if (!STI.getInstrInfo()->verifyInstruction(*MI, Err)) { Index: test/CodeGen/AMDGPU/anonymous-gv.ll =================================================================== --- /dev/null +++ test/CodeGen/AMDGPU/anonymous-gv.ll @@ -0,0 +1,17 @@ +; RUN: llc < %s -mtriple=amdgcn--amdhsa -mcpu=fiji | FileCheck %s + +; Make sure we don't crash on a global variable with no name. +@0 = external addrspace(1) global i32 + +; CHECK-LABEL: {{^}}test: +; CHECK: s_endpgm +define void @test() { + store i32 1, i32 addrspace(1)* @0 + ret void +} + +; CHECK-LABEL: .globl +; CHECK: s_endpgm +define void @1() { + ret void +}