[PATCH] D105042: [SLP][COST][X86]Improve cost model for masked gather.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 22 14:58:26 PDT 2021


ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:4094
             Instruction::Load, VecTy, cast<LoadInst>(VL0)->getPointerOperand(),
             /*VariableMask=*/false, alignment, CostKind, VL0);
       }
----------------
vtjnash wrote:
> Was this supposed to be `CommonAlignment` here? It seems like that added computation is currently unused
Yes, missed it, will fix, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105042



More information about the llvm-commits mailing list