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

[Sam Parker via Phabricator via llvm-commits]

samparker updated this revision to Diff 265455.
samparker added reviewers: Ayal, jsji, hfinkel, gilr.
samparker added a subscriber: dmgreen.
samparker added a comment.
Herald added a subscriber: nemanjai.

Now rebased against master. This is making some (surprising to me) changes to vector code which I can only assume is due to the concerns which @dmgreen raised in D78922 <https://reviews.llvm.org/D78922>. I tried a heavy-handed approach of not using TLI calls for vector casts but that didn't look like a suitable option. So it would be really good to hear from the X86 and PPC guys on whether the vector code changes look sane or whether TLI is just causing more confusion.


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

https://reviews.llvm.org/D78937

Files:
  llvm/include/llvm/CodeGen/BasicTTIImpl.h
  llvm/test/Transforms/LoopVectorize/PowerPC/reg-usage.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.265455.patch
Type: text/x-patch
Size: 19704 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200521/e17c737b/attachment.bin>