[PATCH] D111546: [X86][LV] X86 does *not* prefer vectorized addressing

[Pengfei Wang via Phabricator via llvm-commits]

pengfei added a comment.

In D111546#3065618 <https://reviews.llvm.org/D111546#3065618>, @craig.topper wrote:

> I think this makes sense.
>
> @pengfei do you want to check this on Intel's benchmarks?

Yes, I can check our internal benchmark. I guess this only affect non AVX512 targets, right?



================
Comment at: llvm/lib/Target/X86/X86TargetTransformInfo.cpp:5017-5019
+bool X86TTIImpl::supportsEfficientVectorElementLoadStore() const {
+  return false;
+}
----------------
Do we need to override here? I think it returns false by default.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111546