[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