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

[Chandler Carruth]

On Fri, Jun 8, 2012 at 11:58 AM, Manman Ren <mren at apple.com> wrote:

> Author: mren
> Date: Fri Jun  8 13:58:26 2012
> New Revision: 158220
>
> URL: http://llvm.org/viewvc/llvm-project?rev=158220&view=rev
> Log:
> Enable optimization for integer ABS on X86 if Subtarget has CMOV.
>

Can you include a trivial test case by specifying i386 target CPU?


>
> Modified:
>    llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
>
> Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=158220&r1=158219&r2=158220&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
> +++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Fri Jun  8 13:58:26 2012
> @@ -14548,9 +14548,11 @@
>   if (DCI.isBeforeLegalizeOps())
>     return SDValue();
>
> -  SDValue RV = performIntegerAbsCombine(N, DAG);
> -  if (RV.getNode())
> -    return RV;
> +  if (Subtarget->hasCMov()) {
> +    SDValue RV = performIntegerAbsCombine(N, DAG);
> +    if (RV.getNode())
> +      return RV;
> +  }
>
>   // Try forming BMI if it is available.
>   if (!Subtarget->hasBMI())
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120608/be5f48cf/attachment.html>