[PATCH] D42607: [LoopStrengthReduce, x86] don't add cost for a cmp that will be macro-fused (PR35681)

[Sanjay Patel via Phabricator via llvm-commits]

spatel updated this revision to Diff 132009.
spatel marked 2 inline comments as done.
spatel added a comment.

Patch updated (no code changes; improved the new test file based on Quentin's feedback):

1. Give the IR values real names, so the test is easier to understand.
2. Add IR output testing, so we see what that difference looks like independent of anything else in the x86 backend.


https://reviews.llvm.org/D42607

Files:
  include/llvm/Analysis/TargetTransformInfo.h
  include/llvm/Analysis/TargetTransformInfoImpl.h
  lib/Analysis/TargetTransformInfo.cpp
  lib/Target/X86/X86TargetTransformInfo.cpp
  lib/Target/X86/X86TargetTransformInfo.h
  lib/Transforms/Scalar/LoopStrengthReduce.cpp
  test/CodeGen/X86/rdrand.ll
  test/Transforms/LoopStrengthReduce/X86/ivchain-X86.ll
  test/Transforms/LoopStrengthReduce/X86/macro-fuse-cmp.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42607.132009.patch
Type: text/x-patch
Size: 12115 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180130/5382caf2/attachment.bin>