[llvm] [NVVM] mark math intrinsics as speculatable, default-attrs (PR #94481)

Artem Belevich via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 6 10:21:39 PDT 2024


================
@@ -691,51 +691,51 @@ let TargetPrefix = "nvvm" in {
 
   def int_nvvm_div_approx_ftz_f : ClangBuiltin<"__nvvm_div_approx_ftz_f">,
       DefaultAttrsIntrinsic<[llvm_float_ty], [llvm_float_ty, llvm_float_ty],
-        [IntrNoMem]>;
+        [IntrNoMem, IntrSpeculatable]>;
----------------
Artem-B wrote:

We actually do have `getIntrinsicInstrCost`, it's NVPTX that does not have it implemented yet.

If we combine this patch with an appropriately set intrinsic cost, it should be fine.

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


More information about the llvm-commits mailing list