[PATCH] D96536: Make sure some types are indeed trivially_copyable per llvm::is_trivially_copyable

Danila Malyutin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 15 07:11:18 PST 2021


danilaml added a comment.

I don't know whether these checks are enough (have trivial copy constructor =/=> trivially copyable AFAIK), but otherwise LGTM.



================
Comment at: llvm/unittests/Support/TypeTraitsTest.cpp:37
   // On compilers with support for the standard traits, make sure they agree.
   static_assert(std::is_trivially_copy_constructible<T>::value ==
                     IsTriviallyCopyConstructible,
----------------
I guess these shouldn't have the same problem as trivially_copyable.


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

https://reviews.llvm.org/D96536



More information about the llvm-commits mailing list