[PATCH] D23446: [X86] Enable setcc to srl(ctlz) transformation on btver2 architectures.

pierre gousseau via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 10:21:41 PDT 2016


pgousseau updated this revision to Diff 68209.
pgousseau added a comment.

Rebased changes and following Sanjay's comments:

- Move down feature declaration, add hyphen, add comment.
- Remove hasInterestingUses check.
- Use fast-lzcnt in test and add comments.

Looking again the codegen of openssl without the "hasInterestingUses" constraint the codegen does not seem worse in terms of speed, only the size is not as good as it could be but I think it is ok? Something must have gone wrong during my initial testing I suppose ...


https://reviews.llvm.org/D23446

Files:
  include/llvm/Target/TargetLowering.h
  lib/CodeGen/SelectionDAG/TargetLowering.cpp
  lib/Target/PowerPC/PPCISelLowering.cpp
  lib/Target/X86/X86.td
  lib/Target/X86/X86ISelLowering.cpp
  lib/Target/X86/X86ISelLowering.h
  lib/Target/X86/X86InstrInfo.td
  lib/Target/X86/X86Subtarget.cpp
  lib/Target/X86/X86Subtarget.h
  test/CodeGen/X86/lzcnt-zext-cmp.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23446.68209.patch
Type: text/x-patch
Size: 9642 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160816/cc914118/attachment.bin>


More information about the llvm-commits mailing list