Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
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.
Comment Actions
I don't think we should support differing types for ISD::VP_COPYSIGN. It was disabled for vector ISD::FCOPYSIGN in February 2021 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.