[PATCH] D13850: Changed interface isLegalMaskedLoad() / isLegalMaskedStore()

Elena Demikhovsky via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 18 05:44:59 PDT 2015


delena created this revision.
delena added reviewers: hfinkel, mkuper.
delena added a subscriber: llvm-commits.
delena set the repository for this revision to rL LLVM.

I removed parameter "Consecutive" from  isLegalMaskedLoad() / isLegalMaskedStore().
Originally I planned to use the same interface for masked gather/scatter and set isConsecutive to "false" in this case.

Now I'm implementing masked gather/scatter and see that the interface is inconvenient. I want to add interfaces isLegalMaskedGather() / isLegalMaskedScatter() instead of using the "Consecutive" parameter in the existing interfaces.

Repository:
  rL LLVM

http://reviews.llvm.org/D13850

Files:
  ../include/llvm/Analysis/TargetTransformInfo.h
  ../include/llvm/Analysis/TargetTransformInfoImpl.h
  ../lib/Analysis/TargetTransformInfo.cpp
  ../lib/CodeGen/CodeGenPrepare.cpp
  ../lib/Target/X86/X86TargetTransformInfo.cpp
  ../lib/Target/X86/X86TargetTransformInfo.h
  ../lib/Transforms/Vectorize/LoopVectorize.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13850.37704.patch
Type: text/x-patch
Size: 8716 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151018/8601ba60/attachment.bin>


More information about the llvm-commits mailing list