[llvm] r321991 - [DAGCombine] Fix for PR35761

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 07:51:53 PST 2018


Merged to 6.0 in r322670.

On Mon, Jan 8, 2018 at 2:21 PM, Sam Parker via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: sam_parker
> Date: Mon Jan  8 05:21:24 2018
> New Revision: 321991
>
> URL: http://llvm.org/viewvc/llvm-project?rev=321991&view=rev
> Log:
> [DAGCombine] Fix for PR35761
>
> I had falsely assumed that constant operands would be operand(1) of
> the bin ops that may need their constant operand to be masked.
>
> Bugzilla: https://bugs.llvm.org/show_bug.cgi?id=35761
>
> Differential Revision: https://reviews.llvm.org/D41667
>
> Added:
>     llvm/trunk/test/CodeGen/X86/pr35761.ll
> Modified:
>     llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
>
> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp?rev=321991&r1=321990&r2=321991&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)
> +++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Mon Jan  8 05:21:24 2018
> @@ -3923,10 +3923,16 @@ bool DAGCombiner::BackwardsPropagateMask
>
>      // Narrow any constants that need it.
>      for (auto *LogicN : NodesWithConsts) {
> -      auto *C = cast<ConstantSDNode>(LogicN->getOperand(1));
> -      SDValue And = DAG.getNode(ISD::AND, SDLoc(C), C->getValueType(0),
> -                                SDValue(C, 0), MaskOp);
> -      DAG.UpdateNodeOperands(LogicN, LogicN->getOperand(0), And);
> +      SDValue Op0 = LogicN->getOperand(0);
> +      SDValue Op1 = LogicN->getOperand(1);
> +
> +      if (isa<ConstantSDNode>(Op0))
> +          std::swap(Op0, Op1);
> +
> +      SDValue And = DAG.getNode(ISD::AND, SDLoc(Op1), Op1.getValueType(),
> +                                Op1, MaskOp);
> +
> +      DAG.UpdateNodeOperands(LogicN, Op0, And);
>      }
>
>      // Create narrow loads.
>
> Added: llvm/trunk/test/CodeGen/X86/pr35761.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/pr35761.ll?rev=321991&view=auto
> ==============================================================================
> --- llvm/trunk/test/CodeGen/X86/pr35761.ll (added)
> +++ llvm/trunk/test/CodeGen/X86/pr35761.ll Mon Jan  8 05:21:24 2018
> @@ -0,0 +1,36 @@
> +; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
> +; RUN: llc -mtriple=x86_64-unknown-linux %s -o - | FileCheck %s
> +
> + at x = global i8 0, align 1
> + at y = global i32 0, align 4
> + at z = global i24 0, align 4
> +
> +define void @PR35761(i32 %call) {
> +; CHECK-LABEL: PR35761:
> +; CHECK:       # %bb.0: # %entry
> +; CHECK-NEXT:    movzbl {{.*}}(%rip), %eax
> +; CHECK-NEXT:    andl $1, %eax
> +; CHECK-NEXT:    movzbl {{.*}}(%rip), %ecx
> +; CHECK-NEXT:    xorl $255, %ecx
> +; CHECK-NEXT:    orl %eax, %ecx
> +; CHECK-NEXT:    movw %cx, {{.*}}(%rip)
> +; CHECK-NEXT:    movb $0, z+{{.*}}(%rip)
> +; CHECK-NEXT:    retq
> +entry:
> +  %0 = load i8, i8* @x, align 1
> +  %tobool = trunc i8 %0 to i1
> +  %conv = zext i1 %tobool to i32
> +  %or = or i32 32767, %call
> +  %neg = xor i32 %or, -1
> +  %neg1 = xor i32 %neg, -1
> +  %1 = load i32, i32* @y, align 4
> +  %xor = xor i32 %neg1, %1
> +  %or2 = or i32 %conv, %xor
> +  %conv3 = trunc i32 %or2 to i8
> +  %bf.load = load i24, i24* @z, align 4
> +  %2 = zext i8 %conv3 to i24
> +  %bf.value = and i24 %2, 4194303
> +  store i24 %bf.value, i24* @z, align 2
> +  ret void
> +}
> +
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list