[PATCH] D42323: [DAGCombiner] filter out denorm inputs when calculating sqrt estimate (PR34994)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 1 08:37:30 PST 2018


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D42323#994883, @spatel wrote:

> The patch was approved on the mailing list by @evandro , but he noted that AArch64 fails to pick up this change. That's because it duplicated the masking code as part of custom lowering for that target. I'll add a FIXME for that problem, commit this, and look for a fix as a follow-up.


LGTM - thanks


https://reviews.llvm.org/D42323





More information about the llvm-commits mailing list