[PATCH] D88295: [Sema] Fix volatile check when test if a return object can be implicitly move

Yang Fan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 29 21:44:50 PDT 2020


nullptr.cpp added a comment.

In D88295#2358845 <https://reviews.llvm.org/D88295#2358845>, @aaronpuchert wrote:

> Could you perhaps integrate this into the existing test `clang/test/CXX/special/class.copy/implicit-move.cpp` instead? 
> Whenever you have something that closely corresponds to the standard, CXX is probably the right place for a test.

`clang/test/CXX/special/class.copy/implicit-move.cpp` and `clang/test/CXX/special/class.copy/implicit-move-def.cpp` are tests for implicit (non-)declaration of move constructor and assignment, they have nothing to do with this patch.
Because `copy/move elision` is defined in section [class.copy.elision] in standard, so I put the test in `clang/test/CXX/special/class.copy/copy-elision.cpp`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88295



More information about the cfe-commits mailing list