From f60479b7562957067539c4068677e91df59e9e64 Mon Sep 17 00:00:00 2001 From: Matt Arsenault Date: Tue, 21 Oct 2014 21:08:36 +0000 Subject: [PATCH] R600: Use default GlobalDirective The overridden one wasn't inserting a space, so you would end up with .globalfoo llvm-svn: 220329 --- lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp | 1 - test/CodeGen/R600/global-directive.ll | 14 ++++++++++++++ 2 files changed, 14 insertions(+), 1 deletion(-) create mode 100644 test/CodeGen/R600/global-directive.ll diff --git a/lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp b/lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp index 93b9e90bd53..5393504707c 100644 --- a/lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp +++ b/lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp @@ -33,7 +33,6 @@ AMDGPUMCAsmInfo::AMDGPUMCAsmInfo(StringRef &TT) : MCAsmInfoELF() { UsesELFSectionDirectiveForBSS = true; //===--- Global Variable Emission Directives --------------------------===// - GlobalDirective = ".global"; HasAggressiveSymbolFolding = true; COMMDirectiveAlignmentIsInBytes = false; HasDotTypeDotSizeDirective = false; diff --git a/test/CodeGen/R600/global-directive.ll b/test/CodeGen/R600/global-directive.ll new file mode 100644 index 00000000000..d1244b895c9 --- /dev/null +++ b/test/CodeGen/R600/global-directive.ll @@ -0,0 +1,14 @@ +; RUN: llc -march=r600 -mcpu=SI -verify-machineinstrs < %s | FileCheck -check-prefix=SI %s + +; Make sure the GlobalDirective isn't merged with the function name + +; SI: .globl foo +; SI: {{^}}foo: +define void @foo(i32 addrspace(1)* %out, i32 addrspace(1)* %in) { + %b_ptr = getelementptr i32 addrspace(1)* %in, i32 1 + %a = load i32 addrspace(1)* %in + %b = load i32 addrspace(1)* %b_ptr + %result = add i32 %a, %b + store i32 %result, i32 addrspace(1)* %out + ret void +}