[Mlir-commits] [mlir] [mlir][acc] Fix extraneous space when printing acc.loop (PR #137839)

Valentin Clement バレンタイン クレメン llvmlistbot at llvm.org
Tue Apr 29 11:07:57 PDT 2025


================
@@ -2197,9 +2197,9 @@ def OpenACC_LoopOp : OpenACC_Op<"loop",
 
   let hasCustomAssemblyFormat = 1;
   let assemblyFormat = [{
-    custom<CombinedConstructsLoop>($combined)
     oilist(
-        `gang` `` custom<GangClause>($gangOperands, type($gangOperands),
+        `combined` `(` custom<CombinedConstructsLoop>($combined) `)`
----------------
clementval wrote:

I think we discussed that in the past but we wanted to keep combined information to be the first one and not being lost in with the clauses. This change will allow it to be anywhere. 

Have you try to add `` before the custom parser? That the official assembly format way to remove added space. 
```
`` custom<CombinedConstructsLoop>($combined)
```


https://github.com/llvm/llvm-project/pull/137839


More information about the Mlir-commits mailing list