[PATCH] D32706: [AArch64] Consider widening instructions in cast cost calculation

Matthew Simpson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 2 11:52:24 PDT 2017


mssimpso updated this revision to Diff 97481.
mssimpso marked 3 inline comments as done.
mssimpso retitled this revision from "[AArch64] Consider lengthening instructions in cast cost calculation" to "[AArch64] Consider widening instructions in cast cost calculation".
mssimpso edited the summary of this revision.
mssimpso added a comment.

Addressed Kristof's comments.

- Changed wording to refer to "widening" instructions rather than "lengthening" instructions.
- Changed function name to is isWideningFree.
- Updated the logic to consider the W instruction variants in addition to the L variants.
- Added code generation tests along side the cost test cases.
- Added additional tests for the W variants.


https://reviews.llvm.org/D32706

Files:
  lib/Target/AArch64/AArch64TargetTransformInfo.cpp
  lib/Target/AArch64/AArch64TargetTransformInfo.h
  test/Analysis/CostModel/AArch64/free-widening-casts.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32706.97481.patch
Type: text/x-patch
Size: 27294 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170502/175a1b02/attachment.bin>


More information about the llvm-commits mailing list