Quuxplusone added inline comments.

================
Comment at: clang/lib/Sema/SemaStmt.cpp:3226-3227
+    CopyElisionSemanticsKind CESK = CES_Strict;
+    if (getLangOpts().CPlusPlus20) {
+      CESK = CES_ImplicitlyMovableCXX20;
+    } else if (getLangOpts().CPlusPlus11) {
----------------
rsmith wrote:
> P1825 was accepted as a Defect Report (see 
> https://wiki.edg.com/bin/view/Wg21cologne2019/StrawPolls); is there a reason 
> we're not applying this retroactively?
> Move to accept the changes in P1825R0 (Merged wording for P0527R1 and 
> P1155R3) as a Defect Report and apply them to the C++ working paper.

Yipes. @rsmith, is the intent that modern compilers should support
```
std::unique_ptr<int> from_rvalue(std::unique_ptr<int>&& x) { return x; }
```
in `-std=c++11` mode? Are Clang, GCC, MSVC, EDG all on the same page here? If 
so, awesome. I just find it a priori unlikely that we got informed consensus on 
that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88220

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D88220: [... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D882... Arthur O'Dwyer via Phabricator via cfe-commits
    • [PATCH] D882... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D882... Arthur O'Dwyer via Phabricator via cfe-commits
    • [PATCH] D882... Matheus Izvekov via Phabricator via cfe-commits

Reply via email to