[clang] [clang] Fix a crash from nested ArrayInitLoopExpr (PR #67722)

[Shafik Yaghmour via cfe-commits]

================
@@ -10977,6 +10987,9 @@ bool ArrayExprEvaluator::VisitArrayInitLoopExpr(const ArrayInitLoopExpr *E) {
         return false;
       Success = false;
     }
+
+    // Make sure we run the destructors too.
+    Scope.destroy();
----------------
shafik wrote:

So we don't have to check the return value? It looks like every other use does not discard the return value.

https://github.com/llvm/llvm-project/pull/67722