rjmccall added a comment.

This seems like a good change, but we should make sure we test cases that may 
previously have been covered by the implicit deletion of all copy/move 
constructors.  That may just be a matter of auditing tests.



================
Comment at: clang/include/clang/Basic/AttrDocs.td:2986
 purpose of calls.
-A class annotated with ``trivial_abi`` can have non-trivial destructors or
-copy/move constructors without automatically becoming non-trivial for the
-purposes of calls. For example:
+A class annotated with trivial_abi can have non-trivial destructors or 
copy/move
+constructors or deleted move and copy constructors without automatically
----------------
zoecarver wrote:
> I need to add back the back-ticks. 
This also needs to be turned into a comma-separated list now that it has three 
clauses.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92361

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to