[OpenACC] Fix 'vector' checking when inside combined construct

For some reason when implementing 'vector' I didn't include switch
entries for the combined constructs.  I audited the rest of the uses of
this pattern, and got it right everywhere else, so I'm not sure why I
missed it here.

Fixes: #140339

Author: erichkeane

clang/lib/Sema/SemaOpenACCClause.cpp

@@ -1271,17 +1271,20 @@ OpenACCClause *SemaOpenACCClauseVisitor::VisitVectorClause(
       switch (SemaRef.getActiveComputeConstructInfo().Kind) {
       case OpenACCDirectiveKind::Invalid:
       case OpenACCDirectiveKind::Parallel:
+      case OpenACCDirectiveKind::ParallelLoop:
         // No restriction on when 'parallel' can contain an argument.
         break;
       case OpenACCDirectiveKind::Serial:
+      case OpenACCDirectiveKind::SerialLoop:
         // GCC disallows this, and there is no real good reason for us to permit
         // it, so disallow until we come up with a use case that makes sense.
         DiagIntArgInvalid(SemaRef, IntExpr, "length", OpenACCClauseKind::Vector,
                           Clause.getDirectiveKind(),
                           SemaRef.getActiveComputeConstructInfo().Kind);
         IntExpr = nullptr;
         break;
-      case OpenACCDirectiveKind::Kernels: {
+      case OpenACCDirectiveKind::Kernels:
+      case OpenACCDirectiveKind::KernelsLoop: {
         const auto *Itr =
             llvm::find_if(SemaRef.getActiveComputeConstructInfo().Clauses,
                           llvm::IsaPred<OpenACCVectorLengthClause>);

clang/test/SemaOpenACC/gh140339.cpp

@@ -0,0 +1,22 @@
+// RUN: %clang_cc1 %s -fopenacc -verify
+
+void foo() {
+#pragma acc parallel loop
+  for (int i = 0; i < 5; ++i) {
+#pragma acc loop vector(1)
+    for(int j = 0; j < 5; ++j);
+  }
+
+#pragma acc serial loop
+  for (int i = 0; i < 5; ++i) {
+    // expected-error@+1{{'length' argument on 'vector' clause is not permitted on a 'loop' construct associated with a 'serial loop' compute construct}}
+#pragma acc loop vector(1)
+    for(int j = 0; j < 5; ++j);
+  }
+
+#pragma acc kernels loop
+  for (int i = 0; i < 5; ++i) {
+#pragma acc loop vector(1)
+    for(int j = 0; j < 5; ++j);
+  }
+}