[PATCH] D31175: Improve TargetTransformInfo::getCFInstrCost()

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 24 07:41:19 PDT 2017


jonpa updated this revision to Diff 92943.
jonpa added a comment.

I tried to handle the three cases per your suggestions.

It seems right to model the extract+scalar compare cost as extraction overhead of a vector of i1. This way, a target can define the i1 extractions to cost (which for SystemZ is 2). This also works well with the CostModel.


https://reviews.llvm.org/D31175

Files:
  lib/Target/SystemZ/SystemZTargetTransformInfo.cpp
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Analysis/CostModel/SystemZ/branch-predicated-vectorized-block.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31175.92943.patch
Type: text/x-patch
Size: 6671 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170324/8ac69246/attachment.bin>


More information about the llvm-commits mailing list