[PATCH] D53205: [FPEnv][NFCI] Convert more BinaryOperator::isFNeg(...) to m_FNeg(...)

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 23 13:05:28 PDT 2018


cameron.mcinally added a comment.

In https://reviews.llvm.org/D53205#1273112, @cameron.mcinally wrote:

> @spatel notice the one test change. That seems like a good change to me, i.e. replace a constant pool load with undef. But, perhaps the CP load is a canonicalization that I don't know about....


Although, the xform doesn't appear to be doing anything now. So maybe the test was there for a reason.


Repository:
  rL LLVM

https://reviews.llvm.org/D53205





More information about the llvm-commits mailing list