[llvm-commits] [llvm] r128627 - in /llvm/trunk: lib/Transforms/InstCombine/InstCombineCompares.cpp test/Transforms/InstCombine/fcmp.ll

[Frits van Bommel]

On Thu, Mar 31, 2011 at 12:12 PM, Benjamin Kramer
<benny.kra at googlemail.com> wrote:
> +  // fcmp (fneg x), (fneg y) -> fcmp x, y
> +  Value *X, *Y;
> +  if (match(Op0, m_FNeg(m_Value(X))) && match(Op1, m_FNeg(m_Value(Y))))
> +    return new FCmpInst(I.getPredicate(), X, Y);

Shouldn't this use I.getSwappedPredicate() instead? (-1 > -2, but 1 < 2)