matrix max element check isn't needed anymore

Author: farzonl

clang/include/clang/Basic/DiagnosticSemaKinds.td

@@ -11089,8 +11089,8 @@ def err_first_argument_to_cwsc_pdtor_call : Error<
 def err_second_argument_to_cwsc_not_pointer : Error<
   "second argument to __builtin_call_with_static_chain must be of pointer type">;
 
-def err_tensor_incorrect_num_elements : Error<
-  "%select{too many|too few}0 elements in %select{vector|matrix}1 %select{initialization|operand}4 (expected %2 elements, have %3)">;
+def err_vector_incorrect_num_elements : Error<
+  "%select{too many|too few}0 elements in vector %select{initialization|operand}3 (expected %1 elements, have %2)">;
 def err_altivec_empty_initializer : Error<"expected initializer">;
 
 def err_vector_incorrect_bit_count : Error<

clang/lib/Sema/SemaInit.cpp

@@ -1912,14 +1912,7 @@ void InitListChecker::CheckMatrixType(const InitializedEntity &Entity,
   }
 
   // HLSL requires exactly NumEltsInit to equal Max initializers.
-  if (NumEltsInit != MaxElts) {
-    if (!VerifyOnly)
-      SemaRef.Diag(IList->getBeginLoc(),
-                   diag::err_tensor_incorrect_num_elements)
-          << (NumEltsInit < MaxElts) << /*matrix*/ 1 << MaxElts << NumEltsInit
-          << /*initialization*/ 0;
-    hadError = true;
-  }
+  assert(NumEltsInit == MaxElts && "NumEltsInit must equal MaxElts");
 }
 
 void InitListChecker::CheckVectorType(const InitializedEntity &Entity,
@@ -2071,9 +2064,9 @@ void InitListChecker::CheckVectorType(const InitializedEntity &Entity,
   if (numEltsInit != maxElements) {
     if (!VerifyOnly)
       SemaRef.Diag(IList->getBeginLoc(),
-                   diag::err_tensor_incorrect_num_elements)
-          << (numEltsInit < maxElements) << /*vector*/ 0 << maxElements
-          << numEltsInit << /*initialization*/ 0;
+                   diag::err_vector_incorrect_num_elements)
+          << (numEltsInit < maxElements) << maxElements << numEltsInit
+          << /*initialization*/ 0;
     hadError = true;
   }
 }