[PATCH] D62077: [InstSimplify] Teach fsub -0.0, (fneg X) ==> X about unary fneg

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In D62077#1508478 <https://reviews.llvm.org/D62077#1508478>, @cameron.mcinally wrote:

> > Semi-independent of this patch, but might want to change that m_AnyZeroFP() to m_ZeroFP() now?
>
> @spatel, I did not make this change with the patch. Did you mean `m_PosZeroFP()` instead of `m_ZeroFP()`? Just making sure there's not a new design decision that I don't know about...


Oops - yes, I meant "m_PosZeroFP()" on the 1st match statement. There's no way that a -0.0 could reach that check if I'm seeing it correctly (this patch took care of that possibility).


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62077/new/

https://reviews.llvm.org/D62077