================ @@ -287,6 +287,27 @@ bool SemaSPIRV::CheckSPIRVBuiltinFunctionCall(const TargetInfo &TI, if (CheckAllArgTypesAreCorrect(&SemaRef, TheCall, llvm::ArrayRef(ChecksArr))) return true; + // Check that first two arguments are vectors of the same type ---------------- spall wrote:
the first two can be scalars as well; they just need to be the same type. So I'd recommend updating this comment to state as such. https://github.com/llvm/llvm-project/pull/147342 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits