[PATCH] D114779: [LV] Remove `LoopVectorizationCostModel::useEmulatedMaskMemRefHack()`

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 9 12:20:34 PST 2022


lebedev.ri added a comment.

Could you point me to the bugreport that you have opened before temporarily reverting this? :)

In D114779#3308717 <https://reviews.llvm.org/D114779#3308717>, @dmgreen wrote:

> OK Cool.
>
>> This is a correctness fix. It will not be at all surprising to learn that
>> some other architecture has started to unintentionally rely on this
>> erroneous behavior instead of implementing a correct and precise cost model.
>
> Sure. I just think that one of the architectures relying on this, at least in places, is X86.

Could you please be more specific?

> I was honestly surprised this didn't hurt performance in more places, I thought it was still pretty important. Hence me asking what kind of performance you had run.



> But it looks from these very tests that it's still needed for some things

Could you please be more specific? :)

> and I got reports of a few more. The X86 cases did seem to be hidden more by other costs, like the cost of a vector mul being 6 :)




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114779



More information about the llvm-commits mailing list