[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