Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[mlir][EmitC] Remove unused attribute from verbatim op #80142

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

simon-camp
Copy link

The uses of the attribute were removed in code review of #79584, but it's definition was inadvertently kept.

@llvmbot
Copy link
Member

llvmbot commented Jan 31, 2024

@llvm/pr-subscribers-mlir-emitc

@llvm/pr-subscribers-mlir

Author: Simon Camphausen (simon-camp)

Changes

The uses of the attribute were removed in code review of #79584, but it's definition was inadvertently kept.


Full diff: https://github.com/llvm/llvm-project/pull/80142.diff

1 Files Affected:

  • (modified) mlir/include/mlir/Dialect/EmitC/IR/EmitC.td (+1-4)
diff --git a/mlir/include/mlir/Dialect/EmitC/IR/EmitC.td b/mlir/include/mlir/Dialect/EmitC/IR/EmitC.td
index ca040f2555e13..ada64f10a1675 100644
--- a/mlir/include/mlir/Dialect/EmitC/IR/EmitC.td
+++ b/mlir/include/mlir/Dialect/EmitC/IR/EmitC.td
@@ -574,10 +574,7 @@ def EmitC_VerbatimOp : EmitC_Op<"verbatim"> {
     ```
   }];
 
-  let arguments = (ins
-    StrAttr:$value,
-    UnitAttr:$trailing_semicolon
-  );
+  let arguments = (ins StrAttr:$value);
   let assemblyFormat = "$value attr-dict";
 }
 

Copy link
Member

@marbre marbre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@simon-camp simon-camp merged commit 121a0ef into llvm:main Jan 31, 2024
5 of 6 checks passed
@simon-camp simon-camp deleted the emitc.verbatim branch January 31, 2024 13:43
mgehre-amd pushed a commit to Xilinx/llvm-project that referenced this pull request Mar 11, 2024
The uses of the attribute were removed in code review of llvm#79584, but
it's definition was inadvertently kept.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants