[flang-commits] [PATCH] D151106: [flang][NFC] Move Array constructor inlined temp management into a utility

[Tom Eccles via Phabricator via flang-commits]

tblah accepted this revision.
tblah added a comment.
This revision is now accepted and ready to land.

LGTM, with a small nit.



================
Comment at: flang/lib/Optimizer/Builder/TemporaryStorage.cpp:99-102
+  // TODO: "copy" would probably be better than assign to ensure there are no
+  // side effects (user assignments, temp, lhs finalization)?
+  // This only makes a difference for derived types, and for now derived types
+  // will use the runtime strategy to avoid any bad behaviors.
----------------
nit: please could you add a TODO checking that no derived types get here, just in case the strategy is changed later and this file is forgotten about.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151106