[flang-commits] [PATCH] D151251: [flang][hlfir] Generate temporary storage in Forall/Where [2/2]

[Tom Eccles via Phabricator via flang-commits]

tblah added inline comments.


================
Comment at: flang/lib/Optimizer/Builder/TemporaryStorage.cpp:215
+      hlfir::loadTrivialScalar(loc, builder, hlfir::Entity{retValueBox});
+  if (valueStaticType == builder.getI1Type())
+    return builder.createConvert(loc, valueStaticType, result);
----------------
How about testing the type you get back against valueStaticType to see if they are different? That way, conversions other than `i1` -> `fir.logical` can be handled.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151251