[PATCH] D29540: Scalarization overhead estimation in getIntrinsicInstrCost() improved

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa updated this revision to Diff 91268.
jonpa added a comment.
Herald added a reviewer: javed.absar.

Thanks for review.

> In any case, a comment here explaining the logic would be good.

Added a comment here - is it clear enough?

I also had to update two more tests for ARM and AArch64 targets. Please have a look and let me know if this is acceptable, before I commit.


https://reviews.llvm.org/D29540

Files:
  include/llvm/Analysis/TargetTransformInfo.h
  include/llvm/Analysis/TargetTransformInfoImpl.h
  include/llvm/CodeGen/BasicTTIImpl.h
  lib/Analysis/CostModel.cpp
  lib/Analysis/TargetTransformInfo.cpp
  lib/Target/X86/X86TargetTransformInfo.cpp
  lib/Target/X86/X86TargetTransformInfo.h
  lib/Transforms/Vectorize/BBVectorize.cpp
  lib/Transforms/Vectorize/LoopVectorize.cpp
  lib/Transforms/Vectorize/SLPVectorizer.cpp
  test/Analysis/CostModel/X86/arith-fp.ll
  test/Transforms/LoopVectorize/AArch64/interleaved_cost.ll
  test/Transforms/LoopVectorize/ARM/interleaved_cost.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29540.91268.patch
Type: text/x-patch
Size: 27692 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170310/6a23093a/attachment.bin>