[PATCH] D38085: Use the basic cost if a GEP is not used as addressing mode

Jun Bum Lim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 3 13:27:01 PDT 2017


junbuml updated this revision to Diff 117572.
junbuml added a comment.

The buildbot failure happened when an user of GEPOperator (GEP ConstantExpr) is a constant.  Now returns TCC_Basic for ConstantExpr as it require one or two instructions. For example in AArch64, adrp and add instruction is required. The test case added should show the case.


https://reviews.llvm.org/D38085

Files:
  include/llvm/Analysis/TargetTransformInfo.h
  include/llvm/Analysis/TargetTransformInfoImpl.h
  include/llvm/CodeGen/BasicTTIImpl.h
  include/llvm/IR/Operator.h
  lib/Analysis/TargetTransformInfo.cpp
  lib/Transforms/Scalar/NaryReassociate.cpp
  lib/Transforms/Scalar/StraightLineStrengthReduce.cpp
  test/Analysis/CostModel/AArch64/gep.ll
  test/Analysis/CostModel/X86/vector_gep.ll
  test/Transforms/SimplifyCFG/SpeculativeExecGepCE.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38085.117572.patch
Type: text/x-patch
Size: 11141 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171003/9654de1d/attachment.bin>


More information about the llvm-commits mailing list