[llvm-commits] [llvm] r158220 - /llvm/trunk/lib/Target/X86/X86ISelLowering.cpp

[Anton Korobeynikov]

> +  if (Subtarget->hasCMov()) {
> +    SDValue RV = performIntegerAbsCombine(N, DAG);
> +    if (RV.getNode())
> +      return RV;
> +  }
It'd be better to have the check *inside* performIntegerAbsCombine.
This way it will be obvious why the check is here.

-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University