[CIR][Asm] Remove duplicated lambda & coroutine attributes (llvm#580)

Do not print in cir.func definition the 'attr { ... }' with coroutine or
lambda attributes since they are already printed before the function
name. Otherwise redundancy breaks a future parsing. Sort the attributes
to be skipped so it is more obvious to see the list of attributes.
Improve the tests to check there are no spurious attributes anymore.

Author: keryell

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

@@ -2025,6 +2025,9 @@ ::mlir::Region *cir::FuncOp::getCallableRegion() {
 void cir::FuncOp::print(OpAsmPrinter &p) {
   p << ' ';
 
+  // When adding a specific keyword here, do not forget to omit it in
+  // printFunctionAttributes below or there will be a syntax error when
+  // parsing
   if (getBuiltin())
     p << "builtin ";
 
@@ -2058,10 +2061,19 @@ void cir::FuncOp::print(OpAsmPrinter &p) {
   function_interface_impl::printFunctionAttributes(
       p, *this,
       // These are all omitted since they are custom printed already.
-      {getSymVisibilityAttrName(), getAliaseeAttrName(),
-       getFunctionTypeAttrName(), getLinkageAttrName(), getBuiltinAttrName(),
-       getNoProtoAttrName(), getGlobalCtorAttrName(), getGlobalDtorAttrName(),
-       getExtraAttrsAttrName()});
+      {
+          getAliaseeAttrName(),
+          getBuiltinAttrName(),
+          getCoroutineAttrName(),
+          getExtraAttrsAttrName(),
+          getFunctionTypeAttrName(),
+          getGlobalCtorAttrName(),
+          getGlobalDtorAttrName(),
+          getLambdaAttrName(),
+          getLinkageAttrName(),
+          getNoProtoAttrName(),
+          getSymVisibilityAttrName(),
+      });
 
   if (auto aliaseeName = getAliasee()) {
     p << " alias(";

clang/test/CIR/CodeGen/coro-task.cpp

@@ -461,7 +461,7 @@ folly::coro::Task<int> byRef(const std::string& s) {
 }
 
 // FIXME: this could be less redundant than two allocas + reloads
-// CHECK: cir.func coroutine @_Z5byRefRKSt6string(%arg0: !cir.ptr<![[StdString]]>
+// CHECK: cir.func coroutine @_Z5byRefRKSt6string(%arg0: !cir.ptr<![[StdString]]> {{.*}}22 extra{{.*}}{
 // CHECK: %[[#AllocaParam:]] = cir.alloca !cir.ptr<![[StdString]]>, {{.*}} ["s", init]
 // CHECK: %[[#AllocaFnUse:]] = cir.alloca !cir.ptr<![[StdString]]>, {{.*}} ["s", init]
 
@@ -478,7 +478,7 @@ folly::coro::Task<void> silly_coro() {
 // Make sure we properly handle OnFallthrough coro body sub stmt and
 // check there are not multiple co_returns emitted.
 
-// CHECK: cir.func coroutine @_Z10silly_corov()
+// CHECK: cir.func coroutine @_Z10silly_corov() {{.*}}22 extra{{.*}}{
 // CHECK: cir.await(init, ready : {
 // CHECK: cir.call @_ZN5folly4coro4TaskIvE12promise_type11return_voidEv
 // CHECK-NOT: cir.call @_ZN5folly4coro4TaskIvE12promise_type11return_voidEv
@@ -490,7 +490,7 @@ folly::coro::Task<int> go1() {
   co_return co_await task;
 }
 
-// CHECK: cir.func coroutine @_Z3go1v()
+// CHECK: cir.func coroutine @_Z3go1v() {{.*}}22 extra{{.*}}{
 // CHECK: %[[#IntTaskAddr:]] = cir.alloca ![[IntTask]], !cir.ptr<![[IntTask]]>, ["task", init]
 
 // CHECK:   cir.await(init, ready : {
@@ -525,16 +525,16 @@ folly::coro::Task<int> go1_lambda() {
   co_return co_await task;
 }
 
-// CHECK: cir.func coroutine lambda internal private @_ZZ10go1_lambdavENK3$_0clEv
-// CHECK: cir.func coroutine @_Z10go1_lambdav()
+// CHECK: cir.func coroutine lambda internal private @_ZZ10go1_lambdavENK3$_0clEv{{.*}}22 extra{{.*}}{
+// CHECK: cir.func coroutine @_Z10go1_lambdav() {{.*}}22 extra{{.*}}{
 
 folly::coro::Task<int> go4() {
   auto* fn = +[](int const& i) -> folly::coro::Task<int> { co_return i; };
   auto task = fn(3);
   co_return co_await std::move(task);
 }
 
-// CHECK: cir.func coroutine @_Z3go4v()
+// CHECK: cir.func coroutine @_Z3go4v() {{.*}}22 extra{{.*}}{
 
 // CHECK:   cir.await(init, ready : {
 // CHECK:   }, suspend : {