[PATCH] D55625: [Intrinsic] Unsigned Fixed Point Multiplication Intrinsic
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 16 01:54:39 PST 2019
RKSimon added inline comments.
================
Comment at: llvm/test/CodeGen/X86/umul_fix.ll:262
+
+define <4 x i32> @vec2(<4 x i32> %x, <4 x i32> %y) nounwind {
+; X64-LABEL: vec2:
----------------
leonardchan wrote:
> RKSimon wrote:
> > Can't you keep the vector codegen from scalarizing? See many of the other TargetLowering::expand* instructions which check for legal/custom vector operations before unrolling.
> Could you clarify what you mean by keeping codegen from scalarizing? Vectorization isn't that high of a priority for us, but it would be nice to allow for accepting and returning vectors.
Look at VectorLegalizer::ExpandAddSubSat + TargetLowering::expandAddSubSat - that does something very similar to what I'm suggesting.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55625/new/
https://reviews.llvm.org/D55625
More information about the llvm-commits
mailing list