[flang-commits] [PATCH] D142342: [flang][hlfir] Add hlfir.copy_in and hlfir.copy_out operations

Jean Perier via Phabricator via flang-commits flang-commits at lists.llvm.org
Tue Jan 24 06:51:27 PST 2023


jeanPerier updated this revision to Diff 491760.
jeanPerier marked 4 inline comments as done.
jeanPerier added a comment.

Simplify the operation and answers comments:

- Instead of indicating with a flag how a variable can be optional (null box or null address in box), pass an extra boolean value when the input is optional to indicate if it is present.
- Instead of allowing the operation to return a raw address of box, always returns a box: the box is always created, and it is needed anway when there is a copy-back.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142342/new/

https://reviews.llvm.org/D142342

Files:
  flang/include/flang/Optimizer/Dialect/FIRTypes.td
  flang/include/flang/Optimizer/HLFIR/HLFIROps.td
  flang/lib/Optimizer/HLFIR/IR/HLFIROps.cpp
  flang/test/HLFIR/copy-in-out.fir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D142342.491760.patch
Type: text/x-patch
Size: 6159 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/flang-commits/attachments/20230124/d13c0d24/attachment-0001.bin>


More information about the flang-commits mailing list