================ @@ -2446,37 +2376,41 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned BuiltinID, CallExpr *TheCall) { case Builtin::BI__builtin_hlsl_any: { if (SemaRef.checkArgCount(TheCall, 1)) return true; + if (CheckAnyScalarOrVector(&SemaRef, TheCall, 0)) + return true; break; } case Builtin::BI__builtin_hlsl_asdouble: { if (SemaRef.checkArgCount(TheCall, 2)) return true; - if (CheckUnsignedIntRepresentation(&SemaRef, TheCall)) + if (CheckScalarOrVector(&SemaRef, TheCall, SemaRef.Context.UnsignedIntTy, + 0)) // only check for uint + return true; + if (CheckScalarOrVector(&SemaRef, TheCall, SemaRef.Context.UnsignedIntTy, + 1)) // only check for uint ---------------- farzonl wrote:
or remove the comment about line 2386 so you aren't duplicating the comment on line 2387 and 2390 https://github.com/llvm/llvm-project/pull/138429 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits