Updates for review comments:

- Added '{' and '}' to else with single statement per coding standards.
- BalancedDelimiterTracker() definition moved to where it's used.

Author: ddpagan

clang/lib/AST/OpenMPClause.cpp

@@ -2255,8 +2255,9 @@ void OMPClausePrinter::VisitOMPAllocateClause(OMPAllocateClause *Node) {
       OS << "(";
       Allocator->printPretty(OS, nullptr, Policy, 0);
       OS << ")";
-    } else
+    } else {
       Allocator->printPretty(OS, nullptr, Policy, 0);
+    }
     OS << ":";
     VisitOMPClauseList(Node, ' ');
   } else {

clang/lib/Parse/ParseOpenMP.cpp

@@ -4539,8 +4539,6 @@ bool Parser::ParseOpenMPVarList(OpenMPDirectiveKind DKind,
   bool NeedRParenForLinear = false;
   BalancedDelimiterTracker LinearT(*this, tok::l_paren,
                                    tok::annot_pragma_openmp_end);
-  BalancedDelimiterTracker AllocateT(*this, tok::l_paren,
-                                     tok::annot_pragma_openmp_end);
   // Handle reduction-identifier for reduction clause.
   if (Kind == OMPC_reduction || Kind == OMPC_task_reduction ||
       Kind == OMPC_in_reduction) {
@@ -4807,6 +4805,8 @@ bool Parser::ParseOpenMPVarList(OpenMPDirectiveKind DKind,
       if (Modifier == OMPC_ALLOCATE_allocator) {
         Data.AllocClauseModifier = Modifier;
         ConsumeToken();
+        BalancedDelimiterTracker AllocateT(*this, tok::l_paren,
+                                           tok::annot_pragma_openmp_end);
         if (Tok.is(tok::l_paren)) {
           AllocateT.consumeOpen();
           Tail = ParseAssignmentExpression();