[PATCH] D78937: [CostModel] Use isExtLoad in BasicTTI

[Sam Parker via Phabricator via llvm-commits]

samparker updated this revision to Diff 271627.
samparker added a comment.
Herald added a subscriber: hiraditya.

I've tried to address @dmgreen comments about TLI being queried for an instruction which doesn't match the queried types, so this is now checked. But this wasn't good for X86, as a lot of SLP tests remained scalars, so I've added the TLI check into the X86 backend too. This also reverts the previous test change for PPC.


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

https://reviews.llvm.org/D78937

Files:
  llvm/include/llvm/CodeGen/BasicTTIImpl.h
  llvm/lib/Target/X86/X86TargetTransformInfo.cpp
  llvm/test/Analysis/CostModel/ARM/cast.ll
  llvm/test/Transforms/LoopVectorize/X86/vectorization-remarks-loopid-dbg.ll
  llvm/test/Transforms/LoopVectorize/X86/vectorization-remarks.ll
  llvm/test/Transforms/SLPVectorizer/X86/sext.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78937.271627.patch
Type: text/x-patch
Size: 64580 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200618/43e11ec2/attachment-0001.bin>