================
@@ -19812,11 +19821,13 @@ bool Sema::SemaBuiltinElementwiseTernaryMath(CallExpr
*TheCall) {
Args[I] = Converted.get();
}
- int ArgOrdinal = 1;
- for (Expr *Arg : Args) {
- if (checkFPMathBuiltinElementType(*this, Arg->getBeginLoc(),
Arg->getType(),
- ArgOrdinal++))
- return true;
+ if (enforceFloatingPointCheck) {
+ int ArgOrdinal = 1;
+ for (Expr *Arg : Args) {
+ if (checkFPMathBuiltinElementType(*this, Arg->getBeginLoc(),
+ Arg->getType(), ArgOrdinal++))
----------------
farzonl wrote:
The impact is it just changed the error when we use bool
Before change:
`// expected-error@-1 {{arguments are of different types ('double' vs 'bool')}}`
After change:
`// expected-error@-1 {{3rd argument must be a vector, integer or floating
point type (was 'bool')}}`
https://github.com/llvm/llvm-project/pull/83826
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
