[flang-commits] [flang] [flang][NFC] Add missing sourceBox documentation in embox ops (PR #101221)
via flang-commits
flang-commits at lists.llvm.org
Tue Jul 30 12:01:40 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-flang-fir-hlfir
Author: Valentin Clement (バレンタイン クレメン) (clementval)
<details>
<summary>Changes</summary>
Just noticed that `sourceDoc` was not documented in embox operations description. this patch just add the documentation.
---
Full diff: https://github.com/llvm/llvm-project/pull/101221.diff
2 Files Affected:
- (modified) flang/include/flang/Optimizer/CodeGen/CGOps.td (+2)
- (modified) flang/include/flang/Optimizer/Dialect/FIROps.td (+2)
``````````diff
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.
}];
``````````
</details>
https://github.com/llvm/llvm-project/pull/101221
More information about the flang-commits
mailing list