[PATCH] D11531: fix TLI's combineRepeatedFPDivisors interface to return the minimum user threshold

Sanjay Patel spatel at rotateright.com
Mon Jul 27 11:30:58 PDT 2015


spatel created this revision.
spatel added reviewers: hfinkel, chandlerc.
spatel added a subscriber: llvm-commits.

This fix was suggested as part of D11345. 

We can avoid walking the uses of a divisor node if the target doesn't want the combineRepeatedFPDivisors transform in the first place.

This is a NFC-intended patch, but I want to make sure the new hook is named and defined as expected. I thought about returning a single 'unsigned' value and treating '0' as a special case for "don't do this transform", but that seems a bit dirty.

http://reviews.llvm.org/D11531

Files:
  include/llvm/Target/TargetLowering.h
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  lib/Target/AArch64/AArch64ISelLowering.cpp
  lib/Target/AArch64/AArch64ISelLowering.h
  lib/Target/PowerPC/PPCISelLowering.cpp
  lib/Target/PowerPC/PPCISelLowering.h
  lib/Target/X86/X86ISelLowering.cpp
  lib/Target/X86/X86ISelLowering.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11531.30710.patch
Type: text/x-patch
Size: 6314 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150727/57ebd16f/attachment.bin>


More information about the llvm-commits mailing list