Removed default args from ActOnOpenMPOrderedClause function

Author: mdfazlay

clang/include/clang/Sema/SemaOpenMP.h

@@ -1021,10 +1021,9 @@ class SemaOpenMP : public SemaBase {
   OMPClause *ActOnOpenMPClause(OpenMPClauseKind Kind, SourceLocation StartLoc,
                                SourceLocation EndLoc);
   /// Called on well-formed 'nowait' clause.
-  OMPClause *
-  ActOnOpenMPNowaitClause(SourceLocation StartLoc, SourceLocation EndLoc,
-                          SourceLocation LParenLoc = SourceLocation(),
-                          Expr *Condition = nullptr);
+  OMPClause *ActOnOpenMPNowaitClause(SourceLocation StartLoc,
+                                     SourceLocation EndLoc,
+                                     SourceLocation LParenLoc, Expr *Condition);
   /// Called on well-formed 'untied' clause.
   OMPClause *ActOnOpenMPUntiedClause(SourceLocation StartLoc,
                                      SourceLocation EndLoc);

clang/lib/Sema/SemaOpenMP.cpp

@@ -17957,7 +17957,9 @@ OMPClause *SemaOpenMP::ActOnOpenMPClause(OpenMPClauseKind Kind,
     Res = ActOnOpenMPOrderedClause(StartLoc, EndLoc);
     break;
   case OMPC_nowait:
-    Res = ActOnOpenMPNowaitClause(StartLoc, EndLoc);
+    Res = ActOnOpenMPNowaitClause(StartLoc, EndLoc,
+                                  /*LParenLoc=*/SourceLocation(),
+                                  /*Condition=*/nullptr);
     break;
   case OMPC_untied:
     Res = ActOnOpenMPUntiedClause(StartLoc, EndLoc);
@@ -18123,8 +18125,6 @@ OMPClause *SemaOpenMP::ActOnOpenMPNowaitClause(SourceLocation StartLoc,
 
       ValExpr = Val.get();
     }
-  } else {
-    ValExpr = nullptr;
   }
   DSAStack->setNowaitRegion();
   return new (getASTContext())