[PATCH] D142102: [RISCV] Combine FP_TO_INT to vfwcvt/fvncvt
Yeting Kuo via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 19 04:00:09 PST 2023
fakepaper56 added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:9592
+ // their operand types
+ if (!(VT.getScalarSizeInBits() == SrcVT.getScalarSizeInBits() ||
+ IsNarrowing || IsWidening))
----------------
Is it better to use `VT.getScalarSizeInBits() > SrcVT.getScalarSizeInBits() * 2 || VT.getScalarSizeInBits() * 2 < SrcVT.getScalarSizeInBits()` ?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142102/new/
https://reviews.llvm.org/D142102
More information about the llvm-commits
mailing list