[PATCH] D110579: [AMDGPU] Add two new intrinsics to control fp_trunc rounding mode

David Stuttard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 28 01:29:13 PDT 2021


dstuttard added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/llvm.experimental.fptrunc.round.ll:38
+; CHECK-NEXT:    v_cvt_f16_f32_e32 v2, v1
+; CHECK-NEXT:    s_round_mode 0x0
+; CHECK-NEXT:    s_round_mode 0x8
----------------
Is there a reason why we couldn't make this a single s_round_mode 0x8 rather than going via s_round_mode 0x0?



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110579



More information about the llvm-commits mailing list