[PATCH] D134935: [VP][RISCV] Add vp.copysign and RISC-V support.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 30 09:42:45 PDT 2022


craig.topper added a comment.

In D134935#3825988 <https://reviews.llvm.org/D134935#3825988>, @fakepaper56 wrote:

> `ISD::FCOPYSIGN` could have different types of magnitude operand and sign operand. Should `ISD::VP_FOPYSIGN` have same property? My implements does not loose the constraint like other vp intrinsics.

I don't think we should support differing types for ISD::VP_COPYSIGN. It was disabled for vector ISD::FCOPYSIGN in February 2021 D96037 <https://reviews.llvm.org/D96037>  Though it look likes a command line flag was added to DAGCombiner.cpp in August 2022 to allow it to be re-enabled. Using that flag would very likely cause isel failures on RISC-V.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134935



More information about the llvm-commits mailing list