[PATCH] D74729: [FPEnv] Intrinsic for setting rounding mode

Serge Pavlov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 11 06:41:17 PST 2020


sepavloff added a comment.

In D74729#2359928 <https://reviews.llvm.org/D74729#2359928>, @craig.topper wrote:

> In D74729#2358796 <https://reviews.llvm.org/D74729#2358796>, @sepavloff wrote:
>
>> In D74729#2358110 <https://reviews.llvm.org/D74729#2358110>, @craig.topper wrote:
>>
>>> Do we need type legalization support for targets like RISCV where i32 isn't a legal type in SelectionDAG?
>>
>> Lowering of the intrinsic is anyway custom. Default promotion to i64 is OK.
>
> There is no default promotion code. Each opcode that needs to be promoted by type legalization must be handled in LegalizeIntegerTypes.cpp

Handling of SET_ROUNDING on RISCV is implemented in D91242 <https://reviews.llvm.org/D91242>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74729



More information about the llvm-commits mailing list