]> granicus.if.org Git - clang/commitdiff
Debug Info: Remove unnecessary check for dependent array types
authorDavid Blaikie <dblaikie@gmail.com>
Thu, 9 May 2013 20:48:12 +0000 (20:48 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Thu, 9 May 2013 20:48:12 +0000 (20:48 +0000)
This was added, untested (though the relevant crash was tested), in
r128725/PR9600. Removing it doesn't cause failures & nothing I can
imagine could cause this check to ever return 'true' (we should never be
dealing with dependent types here). The subsequent change to check
"isIncompleteType" (r128855/PR9608) makes a lot more sense.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@181542 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/CGDebugInfo.cpp

index f6ee4b8446d2dce014deada90732e31e5456e492..6678aa7198b121c41d4a9844be9211543fce3d6a 100644 (file)
@@ -1585,7 +1585,7 @@ llvm::DIType CGDebugInfo::CreateType(const ArrayType *Ty,
       Align = 0;
     else
       Align = CGM.getContext().getTypeAlign(Ty->getElementType());
-  } else if (Ty->isDependentSizedArrayType() || Ty->isIncompleteType()) {
+  } else if (Ty->isIncompleteType()) {
     Size = 0;
     Align = 0;
   } else {