some more invalid tests and tidy

llvm · Sep 13, 2024 · 03fc0aa · 03fc0aa
1 parent c192b0b
commit 03fc0aa
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/clang/lib/CIR/Dialect/IR/CIRAttrs.cpp b/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) {

diff --git a/clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp b/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);

diff --git a/clang/test/CIR/IR/invalid-annotations.cir b/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 []>} {}