[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