AMDGPU: Set MCAsmInfo::PointerSize

[arsenm]

Summary

The test should need an update / serve as the test forD11969

[rampitec]

LGTM.

[• tstellarAMD]

test/CodeGen/AMDGPU/basic-debug-info.ll

36

Do we need this clang version in the test? Can any of this other metadata be safely dropped?

[arsenm]

Remove extra metadata

[• tstellarAMD]

Do we still need this patch?

[arsenm]

In D11972#299510, @tstellarAMD wrote:

Do we still need this patch?

I think so

test/CodeGen/AMDGPU/basic-debug-info.ll

36

The clang version should be droppable. Most of it probably can be, I'm not exactly sure what the minimum required is

[arsenm]

It seems the .debug_info section isn't emitted anymore. Is that intentional?

[kzhuravl]

In D11972#664174, @arsenm wrote:

It seems the .debug_info section isn't emitted anymore. Is that intentional?

.debug_info section is not emitted because you have emissionKind: 0. Change it to emissionKind: FullDebug to get debug info emitted.

test/CodeGen/AMDGPU/basic-debug-info.ll

8

• add !dbg !4 before {

18

• emissionKind: 0 -> emissionKind: FullDebug
• remove subprograms: !3

22

• missing distinct before DISubprogram
• function: void ()* @basic_debug_info -> unit: !0

[arsenm]

r294031

test/CodeGen/AMDGPU/basic-debug-info.ll

18

This was the real problem. The other problems I managed to fix or otherwise it fails to parse

lib/Target/AMDGPU/MCTargetDesc/AMDGPUMCAsmInfo.cpp

 //===-- MCTargetDesc/AMDGPUMCAsmInfo.cpp - Assembly Info ------------------===//
 //
 //                     The LLVM Compiler Infrastructure
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 /// \file
 //===----------------------------------------------------------------------===//
 
 #include "AMDGPUMCAsmInfo.h"
+#include "llvm/ADT/Triple.h"
 
 using namespace llvm;
+
 AMDGPUMCAsmInfo::AMDGPUMCAsmInfo(const Triple &TT) : MCAsmInfoELF() {
+  PointerSize = (TT.getArch() == Triple::amdgcn) ? 8 : 4;
   HasSingleParameterDotFile = false;
   //===------------------------------------------------------------------===//
   MaxInstLength = 16;
   SeparatorString = "\n";
   CommentString = ";";
   PrivateLabelPrefix = "";
   InlineAsmStart = ";#ASMSTART";
   InlineAsmEnd = ";#ASMEND";

test/CodeGen/AMDGPU/basic-debug-info.ll

+; RUN: llc -march=amdgcn -mtriple=amdgcn-unknown-amdhsa < %s | FileCheck %s
+
+; FIXME: Should be .byte 8 for pointer size
+
+; CHECK-LABEL: {{^}}basic_debug_info:
+; CHECK: .section .debug_info
+; CHECK: .byte  4                               ; Address Size (in bytes)
+define void @basic_debug_info() #0 {
+entry:
+  ret void, !dbg !9
+}
+
+attributes #0 = { nounwind }
+
+!llvm.dbg.cu = !{!0}
+!llvm.module.flags = !{!7, !8}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, isOptimized: false, runtimeVersion: 0, emissionKind: 0, enums: !2, subprograms: !3)
+!1 = !DIFile(filename: "/tmp/basic-debug-info.cl", directory: "/Users/matt/src/llvm/build_debug")
+!2 = !{}
+!3 = !{!4}
+!4 = !DISubprogram(name: "basic_debug_info", scope: !1, file: !1, line: 1, type: !5, isLocal: false, isDefinition: true, scopeLine: 2, isOptimized: false, function: void ()* @basic_debug_info, variables: !2)
+!5 = !DISubroutineType(types: !6)
+!6 = !{null}
+!7 = !{i32 2, !"Dwarf Version", i32 4}
+!8 = !{i32 2, !"Debug Info Version", i32 3}
+!9 = !DILocation(line: 4, column: 1, scope: !4)