No longer checking the args for the builtin, since the declaration no longer has any arguments

Author: adam-yang

clang/lib/Sema/SemaHLSL.cpp

@@ -2240,11 +2240,6 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned BuiltinID, CallExpr *TheCall) {
       return true;
     }
   }
-  case Builtin::BI__builtin_hlsl_group_memory_barrier_with_group_sync: {
-    if (SemaRef.checkArgCountAtMost(TheCall, 0))
-      return true;
-    break;
-  }
   }
   return false;
 }

clang/test/SemaHLSL/BuiltIns/GroupMemoryBarrierWithGroupSync-errors.hlsl

@@ -2,5 +2,5 @@
 
 void test_too_many_arg() {
   __builtin_hlsl_group_memory_barrier_with_group_sync(0);
-  // expected-error@-1 {{too many arguments to function call, expected at most 0, have 1}}
+  // expected-error@-1 {{too many arguments to function call, expected 0, have 1}}
 }