DebugInfo: Simplified Template Names: drop unneeded space in arrays

Matching a recent clang change I've made, now 'int[3]' is formatted
without the space between the type and array bound. This commit updates
libDebugInfoDWARF/llvm-dwarfdump to match that formatting.

Author: dwblaikie

cross-project-tests/debuginfo-tests/clang_llvm_roundtrip/simplified_template_names.cpp

@@ -185,7 +185,7 @@ int main() {
   f1<int &>();
   f1<int &&>();
   f1<const int>();
-  // f1<int[3]>();
+  f1<int[3]>();
   f1<void>();
   f1<outer_class::inner_class>();
   f1<unsigned long>();
@@ -291,7 +291,7 @@ int main() {
   f1<int *const(&)[1]>();
   f1<void (::udt::* const&)()>();
   f1<void (*(int))(float)>();
-  // f1<t1<int>[1]>();
+  f1<t1<int>[1]>();
   f1<void (*)() noexcept>();
   f1<void (decltype(A))>();
   struct t8 { decltype(A) m; };

llvm/lib/DebugInfo/DWARF/DWARFDie.cpp

@@ -226,9 +226,6 @@ struct DWARFTypePrinter {
     }
     case DW_TAG_array_type: {
       appendQualifiedNameBefore(Inner);
-      if (Word)
-        OS << ' ';
-      Word = false;
       break;
     }
     case DW_TAG_reference_type:

llvm/test/DebugInfo/X86/tu-to-non-named-type.ll

@@ -11,7 +11,7 @@
 ; CHECK-NOT: {{DW_TAG|NULL}}
 ; CHECK:     DW_TAG_member
 ; CHECK-NEXT:       DW_AT_name    ("x")
-; CHECK-NEXT:       DW_AT_type    ({{.*}} "int [1]"
+; CHECK-NEXT:       DW_AT_type    ({{.*}} "int[1]"
 
 ; But make sure we still use a type unit for an anonymous type that still has a
 ; name for linkage purposes (due to being defined in a typedef).

llvm/test/tools/llvm-dwarfdump/X86/prettyprint_types.s

@@ -112,7 +112,7 @@
 # CHECK:   DW_AT_type{{.*}}"int *const (&)[1]")
 # CHECK:   DW_AT_type{{.*}}"int *const[1]")
 # CHECK:   DW_AT_type{{.*}}"const int (&)[1]")
-# CHECK:   DW_AT_type{{.*}}"const int [1]")
+# CHECK:   DW_AT_type{{.*}}"const int[1]")
 
 # subroutine types
 # CHECK:   DW_AT_type{{.*}}"int ()"
@@ -129,7 +129,7 @@
 # qualified types
 # CHECK:   DW_AT_type{{.*}}"ns::inner"
 # CHECK:   DW_AT_type{{.*}}"ns::inner ()"
-# CHECK:   DW_AT_type{{.*}}"ns::inner [1]"
+# CHECK:   DW_AT_type{{.*}}"ns::inner[1]"
 # CHECK:   DW_AT_type{{.*}}"ns::inner *"
 # CHECK:   DW_AT_type{{.*}}"ns::inner (ns::inner::*)(ns::inner)"
 # CHECK:   DW_AT_type{{.*}}"const ns::inner"

llvm/test/tools/llvm-dwarfdump/X86/prettyprint_types_handcrafted.s

@@ -7,7 +7,7 @@
 # CHECK: DW_TAG_variable
 
 # array_type
-# CHECK:   DW_AT_type{{.*}}"int{{ }}
+# CHECK:   DW_AT_type{{.*}}"int
 # Testing with a default lower bound of 0 and the following explicit bounds:
 #   lower_bound(1)
 # CHECK-NOT: {{.}}
@@ -34,7 +34,7 @@
 # array_type with a language with a default lower bound of 1 instead of 0 and
 # an upper bound of 2. This describes an array with 2 elements (whereas with a
 # default lower bound of 0 it would be an array of 3 elements)
-# CHECK: DW_AT_type{{.*}}"int [2]"
+# CHECK: DW_AT_type{{.*}}"int[2]"
 
 	.section	.debug_str,"MS",@progbits,1
 .Lint_name:

llvm/test/tools/llvm-dwarfdump/XCOFF/basic.test

@@ -95,7 +95,7 @@
 # DWARF32-NEXT:                    DW_AT_name    ("__func__")
 # DWARF32-NEXT:                    DW_AT_decl_file       ("/basic.c")
 # DWARF32-NEXT:                    DW_AT_decl_line       (0)
-# DWARF32-NEXT:                    DW_AT_type    (0x0000005a "const char [5]")
+# DWARF32-NEXT:                    DW_AT_type    (0x0000005a "const char[5]")
 # DWARF32:       0x00000096:     NULL
 # DWARF32:       0x00000097:   NULL
 # DWARF32:       .debug_line contents:
@@ -225,7 +225,7 @@
 # DWARF64-NEXT:                    DW_AT_name    ("__func__")
 # DWARF64-NEXT:                    DW_AT_decl_file       ("/basic.c")
 # DWARF64-NEXT:                    DW_AT_decl_line       (0)
-# DWARF64-NEXT:                    DW_AT_type    (0x00000076 "const char [5]")
+# DWARF64-NEXT:                    DW_AT_type    (0x00000076 "const char[5]")
 # DWARF64:       0x000000ce:     NULL
 # DWARF64:       0x000000cf:   NULL
 # DWARF64:       .debug_line contents: