[PATCH] D20170: [clang-tidy] TypeTraits - Type is not expensive to copy when it has a deleted copy constructor.

Etienne Bergeron via cfe-commits cfe-commits at lists.llvm.org
Thu May 12 20:18:57 PDT 2016


etienneb added inline comments.

================
Comment at: clang-tidy/utils/TypeTraits.cpp:22
@@ -20,2 +21,3 @@
+
 bool classHasTrivialCopyAndDestroy(QualType Type) {
   auto *Record = Type->getAsCXXRecordDecl();
----------------
My bad! I didn't saw the file change when reading. Forget about it.

================
Comment at: clang-tidy/utils/TypeTraits.cpp:29
@@ +28,3 @@
+
+bool hasDeletedCopyConstructor(QualType Type) {
+  auto *Record = Type->getAsCXXRecordDecl();
----------------
ditto. I was thinking this was in the checker file.
ignore it.


http://reviews.llvm.org/D20170





More information about the cfe-commits mailing list