[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