[PATCH] D110407: [fir] Add fir.save_result op

[Valentin Clement via Phabricator via llvm-commits]

clementval added inline comments.


================
Comment at: flang/lib/Optimizer/Dialect/FIROps.cpp:1305
+static mlir::LogicalResult verify(fir::SaveResultOp op) {
+  auto resultType = op.value().getType();
+  if (resultType != fir::dyn_cast_ptrEleTy(op.memref().getType()))
----------------
mehdi_amini wrote:
> Nit: The naming `resultType` confused me, because this is the input value to the operation and not its result (there aren't any results for this op)
> 
It the resultType of the function as mentioned in the description. I'm not sure we can have a better name for that maybe `valueType`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110407