[PATCH] D107230: [RISCV] Support FP_TO_S/UINT_SAT for i32 and i64.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 7 16:01:21 PDT 2021


craig.topper added a comment.

In D107230#2932937 <https://reviews.llvm.org/D107230#2932937>, @luismarques wrote:

> LGTM.
>
> BTW, in the tablegen files some pre-existing comments like `float->int32` have the width suffix and others don't. I'm not sure if there's any particular logic to that that we should be following?

I'll add the suffixes to the new code entries in this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107230



More information about the llvm-commits mailing list