<p dir="ltr">LGTM</p>
<div class="gmail_quote">On 21 Jan 2015 08:25, "Agustín Bergé" <<a href="mailto:kaballo86@hotmail.com">kaballo86@hotmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi rsmith,<br>
<br>
Fix `isTriviallyCopyableType` for arrays. An array of type `T` is trivially copyable if `T` is trivially copyable.<br>
<br>
<a href="http://reviews.llvm.org/D7096" target="_blank">http://reviews.llvm.org/D7096</a><br>
<br>
Files:<br>
  lib/AST/Type.cpp<br>
  test/SemaCXX/type-traits.cpp<br>
<br>
Index: lib/AST/Type.cpp<br>
===================================================================<br>
--- lib/AST/Type.cpp<br>
+++ lib/AST/Type.cpp<br>
@@ -1089,7 +1089,7 @@<br>
<br>
 bool QualType::isTriviallyCopyableType(ASTContext &Context) const {<br>
   if ((*this)->isArrayType())<br>
-    return Context.getBaseElementType(*this).isTrivialType(Context);<br>
+    return Context.getBaseElementType(*this).isTriviallyCopyableType(Context);<br>
<br>
   if (Context.getLangOpts().ObjCAutoRefCount) {<br>
     switch (getObjCLifetime()) {<br>
Index: test/SemaCXX/type-traits.cpp<br>
===================================================================<br>
--- test/SemaCXX/type-traits.cpp<br>
+++ test/SemaCXX/type-traits.cpp<br>
@@ -1857,6 +1857,9 @@<br>
   { int arr[T(__is_trivially_copyable(HasNonPOD))]; }<br>
   { int arr[T(__is_trivially_copyable(DerivesHasCons))]; }<br>
   { int arr[T(__is_trivially_copyable(DerivesHasRef))]; }<br>
+  { int arr[T(__is_trivially_copyable(NonTrivialDefault))]; }<br>
+  { int arr[T(__is_trivially_copyable(NonTrivialDefault[]))]; }<br>
+  { int arr[T(__is_trivially_copyable(NonTrivialDefault[3]))]; }<br>
<br>
   { int arr[F(__is_trivially_copyable(HasCopyAssign))]; }<br>
   { int arr[F(__is_trivially_copyable(HasMoveAssign))]; }<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
</blockquote></div>