[clang] [llvm] [HLSL] implement `mad` intrinsic (PR #83826)

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 4 09:48:08 PST 2024


================
@@ -19800,7 +19808,8 @@ bool Sema::SemaBuiltinVectorMath(CallExpr *TheCall, QualType &Res) {
   return false;
 }
 
-bool Sema::SemaBuiltinElementwiseTernaryMath(CallExpr *TheCall) {
+bool Sema::SemaBuiltinElementwiseTernaryMath(CallExpr *TheCall,
+                                             bool enforceFloatingPointCheck) {
----------------
bogner wrote:

Formal arguments are capitalized in LLVM's style, like "EnforceFloatingPointCheck". I'm also not a huge fan of this name - it isn't obvious what it means without looking at the implementation of the function.

https://github.com/llvm/llvm-project/pull/83826


More information about the llvm-commits mailing list