some more invalid tests and tidy

Author: ghehg

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

@@ -547,6 +547,7 @@ void MethodAttr::print(AsmPrinter &printer) const {
 //===----------------------------------------------------------------------===//
 // GlobalAnnotationValuesAttr definitions
 //===----------------------------------------------------------------------===//
+
 LogicalResult GlobalAnnotationValuesAttr::verify(
     function_ref<::mlir::InFlightDiagnostic()> emitError,
     mlir::ArrayAttr annotations) {

clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp

@@ -2044,7 +2044,7 @@ class CIRGlobalOpLowering
     // Rewrite op.
     auto llvmGlobalOp = rewriter.replaceOpWithNewOp<mlir::LLVM::GlobalOp>(
         op, llvmType, isConst, linkage, symbol, init.value(),
-        /*alignment*/op.getAlignment().value_or(0),
+        /*alignment*/ op.getAlignment().value_or(0),
         /*addrSpace*/ getGlobalOpTargetAddrSpace(op),
         /*dsoLocal*/ false, /*threadLocal*/ (bool)op.getTlsModelAttr(),
         /*comdat*/ mlir::SymbolRefAttr(), attributes);

clang/test/CIR/IR/invalid-annotations.cir

@@ -1,6 +1,13 @@
 // Test attempt to construct ill-formed global annotations
 // RUN: cir-opt %s -verify-diagnostics -split-input-file
 
+
+// expected-error @below {{invalid kind of attribute specified}}
+// expected-error @below {{failed to parse AnnotationAttr parameter 'name' which is to be a `StringAttr`}}
+cir.global  external @a = #cir.ptr<null> : !cir.ptr<!cir.double> [#cir.annotation<name = 18, args = ["21", 12 : i32]>]
+
+// -----
+
 // expected-error @below {{GlobalAnnotationValuesAttr should at least have one annotation}}
 module attributes {cir.global_annotations = #cir<annotation_values []>} {}