[PATCH] D125937: [NVVM] Update intrinsic defintions to include the `nocallback` attribute

[Joseph Huber via Phabricator via llvm-commits]

jhuber6 added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsNVVM.td:1238
   def int_nvvm_ff2bf16x2_rn : GCCBuiltin<"__nvvm_ff2bf16x2_rn">,
-       Intrinsic<[llvm_i32_ty], [llvm_float_ty, llvm_float_ty], [IntrNoMem]>;
+       Intrinsic<[llvm_i32_ty], [llvm_float_ty, llvm_float_ty], [IntrNoMem, IntrNoCallback]>;
   def int_nvvm_ff2bf16x2_rn_relu : GCCBuiltin<"__nvvm_ff2bf16x2_rn_relu">,
----------------
tra wrote:
> Is there a reason these f32<-> f16/bf16 conversion intrinsics should not be a `DefaultAttrsIntrinsic` ?
They probably could be, I could add them in.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125937/new/

https://reviews.llvm.org/D125937