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

Felix Berger via cfe-commits cfe-commits at lists.llvm.org
Thu May 12 07:23:41 PDT 2016


flx added inline comments.

================
Comment at: clang-tidy/utils/TypeTraits.cpp:32
@@ +31,3 @@
+  auto *Record = Type->getAsCXXRecordDecl();
+  if (Record == nullptr || !Record->hasDefinition())
+    return false;
----------------
aaron.ballman wrote:
> `!Record` instead of explicit comparison.
Does this make a difference for code generation or is this just a convention here for conciseness?


http://reviews.llvm.org/D20170





More information about the cfe-commits mailing list