[PATCH] D11345: ignore duplicate divisor uses when transforming into reciprocal multiplies (PR24141)

Sanjay Patel spatel at rotateright.com
Mon Jul 27 11:39:29 PDT 2015


spatel added a comment.

> 1. Move the DAGCombiner's combineRepeatedFPDivisors logic into a helper function because it's getting too big (NFC).


Checked in:
http://reviews.llvm.org/rL243293

> 2. Fix the TLI hook to return the minimum uses threshold (NFC).


Submitted for review:
http://reviews.llvm.org/D11531

> 3. Fix the data structure to be a SetVector to fix PR24141.


This is a trivial change if the previous change is ok, so I'll abandon this as a patch for review unless someone would like to see the updated patch here prior to commit.


http://reviews.llvm.org/D11345







More information about the llvm-commits mailing list