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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri marked an inline comment as done.
lebedev.ri added a comment.

In D111546#3067965 <https://reviews.llvm.org/D111546#3067965>, @pengfei wrote:

> In D111546#3066133 <https://reviews.llvm.org/D111546#3066133>, @lebedev.ri wrote:
>
>> In D111546#3066122 <https://reviews.llvm.org/D111546#3066122>, @pengfei wrote:
>>
>>> 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.
>>
>> Thank you.
>> I'm not sure how this measures standalone,
>> my main hope this unblocks D111460 <https://reviews.llvm.org/D111460>.
>
> Our internal benchmarks are not impacted by this patch.

Cool! Thanks for checking.
Does anyone want to formally stamp this patch?

> So I don't see evidences it can unblock D111460 <https://reviews.llvm.org/D111460>.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111546