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

Bardia Mahjour via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 8 14:44:25 PST 2022


bmahjour added a comment.

Even though the initial intention of the "hack" might have been to prevent predication on some X86 subtargets, it was added in a way that affected all targets, so it could expose cost model issues on various targets if suddenly removed. I wasn't aware of this and just noticed it....has any performance runs been done on other targets (eg. Power)?


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