[PATCH] D88797: [flang] Fix copy elision assumption.

[Peter Klausler via Phabricator via llvm-commits]

klausler accepted this revision.
klausler added inline comments.
This revision is now accepted and ready to land.


================
Comment at: flang/include/flang/Common/restorer.h:25
 public:
-  explicit Restorer(A &p) : p_{p}, original_{std::move(p)} {}
+  explicit Restorer(A &p, A Original) : p_{p}, original_{std::move(Original)} {}
   ~Restorer() { p_ = std::move(original_); }
----------------
Here, and below: we're using lower-case names in flang/ for variables.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88797