Don't assert if we find a dependently-typed variable in the
redeclaration chain for an array.
A prior attempt to fix this in r280330 didn't handle the case where the
old variable is dependent and the new one is not.
It is notable and worrying that the test case in this example forms a
redeclaration chain for a non-dependent variable that includes a
declaration with a dependent type. We should probably fix that too.
diff --git a/clang/test/SemaTemplate/array-redeclaration.cpp b/clang/test/SemaTemplate/array-redeclaration.cpp
index 4edee70..af0a277 100644
--- a/clang/test/SemaTemplate/array-redeclaration.cpp
+++ b/clang/test/SemaTemplate/array-redeclaration.cpp
@@ -31,3 +31,9 @@
void foo4() {
foo3<5>();
}
+
+namespace NS {
+ int f() { extern int arr[3]; { extern int arr[]; } return 0; }
+ template<typename T> void g() { extern int arr[3]; extern T arr; }
+ template void g<int[]>();
+}