[flang-commits] [flang] [flang][NFC] Add missing sourceBox documentation in embox ops (PR #101221)

Valentin Clement バレンタイン クレメン via flang-commits flang-commits at lists.llvm.org
Tue Jul 30 12:01:13 PDT 2024


https://github.com/clementval created https://github.com/llvm/llvm-project/pull/101221

Just noticed that `sourceDoc` was not documented in embox operations description. this patch just add the documentation. 

>From 5ca9751f1f4bde50bcf86c163b27447f98c4d992 Mon Sep 17 00:00:00 2001
From: Valentin Clement <clementval at gmail.com>
Date: Tue, 30 Jul 2024 12:00:02 -0700
Subject: [PATCH] [flang][NFC] Add missing sourceBox documentation in embox ops

---
 flang/include/flang/Optimizer/CodeGen/CGOps.td  | 2 ++
 flang/include/flang/Optimizer/Dialect/FIROps.td | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/flang/include/flang/Optimizer/CodeGen/CGOps.td b/flang/include/flang/Optimizer/CodeGen/CGOps.td
index f4740a263ffd2..2e07994690e5f 100644
--- a/flang/include/flang/Optimizer/CodeGen/CGOps.td
+++ b/flang/include/flang/Optimizer/CodeGen/CGOps.td
@@ -48,6 +48,8 @@ def fircg_XEmboxOp : fircg_Op<"ext_embox", [AttrSizedOperandSegments]> {
        - substring: A substring operator (offset, length) for CHARACTER.
        - LEN type parameters: A vector of runtime LEN type parameters that
          describe an correspond to the elemental derived type.
+       - sourceBox: A box to read information from such as CFI type,
+         type descriptor or element size to populate the new descriptor.
 
     The memref and shape arguments are mandatory. The rest are optional.
   }];
diff --git a/flang/include/flang/Optimizer/Dialect/FIROps.td b/flang/include/flang/Optimizer/Dialect/FIROps.td
index bee8e8f603ce3..2b09745b49cfd 100644
--- a/flang/include/flang/Optimizer/Dialect/FIROps.td
+++ b/flang/include/flang/Optimizer/Dialect/FIROps.td
@@ -782,6 +782,8 @@ def fir_EmboxOp : fir_Op<"embox", [NoMemoryEffect, AttrSizedOperandSegments]> {
           lower bounds and extents may not be known until runtime),
         - slice: an array section can be described with a slice triple,
         - typeparams: for emboxing a derived type with LEN type parameters,
+        - sourceBox: A box to read information from such as CFI type,
+          type descriptor or element size to populate the new descriptor.
         - accessMap: unused/experimental.
   }];
 



More information about the flang-commits mailing list