[PATCH] D67281: [AArch64][SimplifyCFG] Add additional cost for instructions in mergeConditionalStoreToAddress

[Pavel Kosov via Phabricator via llvm-commits]

kpdev42 updated this revision to Diff 234845.
kpdev42 added a comment.

In lambda IsWorthwile in mergeConditionalStoreToAddress added check for additional instructions which will be generated after `select` lowering
Update test.

File with runtime benchmark: F11110299: small_benchmark.c <https://reviews.llvm.org/F11110299>

On device with CortexA76 patched version better than original in about 2 times

  To = execution time of original version (average for 1000 iterations)
  Tp = execution time of patched version (average for 1000 iterations)
  
                            To / Tp 
  Match all branches.       2.11
  Match no branches.        2.12
  Match IN branch only.     2.12
  Match OUT branch only.    2.11
  Match EX branch only.     2.07
  Match IN & OUT branches.  2.13
  Match IN & EX branches.   2.15
  Match OUT & EX branches.  2.08


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

https://reviews.llvm.org/D67281

Files:
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
  llvm/lib/Analysis/TargetTransformInfo.cpp
  llvm/lib/Target/AArch64/AArch64TargetTransformInfo.h
  llvm/lib/Transforms/Utils/SimplifyCFG.cpp
  llvm/test/Transforms/SimplifyCFG/AArch64/check-convert-to-cond-instr.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67281.234845.patch
Type: text/x-patch
Size: 8313 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191220/4e5c75f9/attachment.bin>