[Mlir-commits] [mlir] 6c7cb55 - [mlir][cf] Fix ControlFlow dialect's operations' summary (#101494)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Fri Aug 2 05:08:50 PDT 2024


Author: Andrey Timonin
Date: 2024-08-02T14:08:45+02:00
New Revision: 6c7cb55f8f8dc833fbd3d0e4261a503cf1e41341

URL: https://github.com/llvm/llvm-project/commit/6c7cb55f8f8dc833fbd3d0e4261a503cf1e41341
DIFF: https://github.com/llvm/llvm-project/commit/6c7cb55f8f8dc833fbd3d0e4261a503cf1e41341.diff

LOG: [mlir][cf] Fix ControlFlow dialect's operations' summary (#101494)

Ensure that the summary of the operation in the TableGen starts with a capital letter.

This is in line with our [guideline](https://mlir.llvm.org/docs/DefiningDialects/Operations/#operation-arguments)
and the current output of the documentation generator which capitalizes the summary.

Added: 
    

Modified: 
    mlir/include/mlir/Dialect/ControlFlow/IR/ControlFlowOps.td

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Dialect/ControlFlow/IR/ControlFlowOps.td b/mlir/include/mlir/Dialect/ControlFlow/IR/ControlFlowOps.td
index 3bdbfb0fedb9c..48f12b46a57f1 100644
--- a/mlir/include/mlir/Dialect/ControlFlow/IR/ControlFlowOps.td
+++ b/mlir/include/mlir/Dialect/ControlFlow/IR/ControlFlowOps.td
@@ -68,7 +68,7 @@ def BranchOp : CF_Op<"br", [
     DeclareOpInterfaceMethods<BranchOpInterface, ["getSuccessorForOperands"]>,
     Pure, Terminator
   ]> {
-  let summary = "branch operation";
+  let summary = "Branch operation";
   let description = [{
     The `cf.br` operation represents a direct branch operation to a given
     block. The operands of this operation are forwarded to the successor block,
@@ -116,7 +116,7 @@ def CondBranchOp : CF_Op<"cond_br",
     [AttrSizedOperandSegments,
      DeclareOpInterfaceMethods<BranchOpInterface, ["getSuccessorForOperands"]>,
      Pure, Terminator]> {
-  let summary = "conditional branch operation";
+  let summary = "Conditional branch operation";
   let description = [{
     The `cf.cond_br` terminator operation represents a conditional branch on a
     boolean (1-bit integer) value. If the bit is set, then the first destination
@@ -231,7 +231,7 @@ def SwitchOp : CF_Op<"switch",
     [AttrSizedOperandSegments,
      DeclareOpInterfaceMethods<BranchOpInterface, ["getSuccessorForOperands"]>,
      Pure, Terminator]> {
-  let summary = "switch operation";
+  let summary = "Switch operation";
   let description = [{
     The `cf.switch` terminator operation represents a switch on a signless integer
     value. If the flag matches one of the specified cases, then the


        


More information about the Mlir-commits mailing list