[Clang] Fix a crash introduced in PR#88666 (llvm#89567)

Change-Id: I0b7fb53576e77dec5c1843a99e89ec466ed63c89

Author: shiltian

clang/lib/Sema/SemaStmtAttr.cpp

@@ -109,7 +109,7 @@ static Attr *handleLoopHintAttr(Sema &S, Stmt *St, const ParsedAttr &A,
     SetHints(LoopHintAttr::Unroll, LoopHintAttr::Disable);
   } else if (PragmaName == "unroll") {
     // #pragma unroll N
-    if (ValueExpr) {
+    if (ValueExpr && !ValueExpr->isValueDependent()) {
       llvm::APSInt ValueAPS;
       ExprResult R = S.VerifyIntegerConstantExpression(ValueExpr, &ValueAPS);
       assert(!R.isInvalid() && "unroll count value must be a valid value, it's "

revert_patches.txt

@@ -9,6 +9,3 @@ Revert :breaks hip catch tests.
 1de7e6c8cba2 [clang] move -Wcast-function-type under -Wextra (#77178)
 999d4f840777 Split -Wcast-function-type into a separate group (#86131)   
 
-Revert; breaks build of rccl and blas and rocSolver
-* f4bbcac2 [Clang] Allow the value of unroll count to be zero  in `#pragma GCC unroll` and `#pragma unroll` (#88666)
-Clang] Fix a crash introduced in PR#88666 (#89567)