[PATCH] D56082: [X86][SLP] Enable SLP vectorization for 128-bit horizontal X86 instructions (add, sub)
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 16 03:07:33 PST 2019
RKSimon added inline comments.
================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:150
+unsigned X86TTIImpl::getMinVectorRegisterBitWidth() const {
+ return 64;
----------------
anton-afanasyev wrote:
> RKSimon wrote:
> > Can we add a IsFloat bool argument here?
> That is not possible: at this stage we can only operate by vector register width notion regardless of the scalars type. And what is the cause we can need it?
Don't refer to MMX - its gives the wrong impression that we actually generate code for it (all we ever do is emit intrinsic code).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56082/new/
https://reviews.llvm.org/D56082
More information about the llvm-commits
mailing list