[CIR][Bugfix] Fix cir.array getTypeSizeInBits method

Constant initialization of static local arrays would fail due to a
mismatch between the variable and the initializer type size. This patch
fixes the data layout interface implementation for the cir.array type.

A complete array in C/C++ should have its type size in bits equal to
the size of the array times the size of the element type.

ghstack-source-id: 56f3f29
Pull Request resolved: #206

Author: sitio-couto

clang/lib/CIR/Dialect/IR/CIRTypes.cpp

@@ -202,7 +202,7 @@ unsigned PointerType::getPreferredAlignment(
 unsigned
 ArrayType::getTypeSizeInBits(const ::mlir::DataLayout &dataLayout,
                              ::mlir::DataLayoutEntryListRef params) const {
-  return dataLayout.getTypeSizeInBits(getEltType());
+  return getSize() * dataLayout.getTypeSizeInBits(getEltType());
 }
 
 unsigned

clang/test/CIR/CodeGen/static-vars.c

@@ -42,3 +42,9 @@ void func3(void) {
   static int *constAddr = &var;
   // CHECK-DAG: cir.global "private" internal @func3.constAddr = #cir.global_view<@func3.var> : !cir.ptr<!s32i>
 }
+
+// Should match type size in bytes between var and initializer.
+void func4(void) {
+  static char string[] = "Hello";
+  // CHECK-DAG: cir.global "private" internal @func4.string = #cir.const_array<"Hello\00" : !cir.array<!s8i x 6>> : !cir.array<!s8i x 6>
+}