[llvm] r342457 - Fixes removal of dead elements from PressureDiff (PR37252).
Yuri Gribov via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 18 07:26:07 PDT 2018
Hi Hans,
Thanks, I'll check this. Any idea how to update email address?
On Tue, Sep 18, 2018 at 3:16 PM Hans Wennborg <hans at chromium.org> wrote:
>
> Also the email address associated with your committer ID seems to be bouncing.
>
> On Tue, Sep 18, 2018 at 4:14 PM, Hans Wennborg <hans at chromium.org> wrote:
> > Reverted in r342482. This broke the lit tests.
> >
> > On Tue, Sep 18, 2018 at 11:53 AM, Yury Gribov via llvm-commits
> > <llvm-commits at lists.llvm.org> wrote:
> >> Author: ygribov
> >> Date: Tue Sep 18 02:53:42 2018
> >> New Revision: 342457
> >>
> >> URL: http://llvm.org/viewvc/llvm-project?rev=342457&view=rev
> >> Log:
> >> Fixes removal of dead elements from PressureDiff (PR37252).
> >>
> >> Reviewed By: MatzeB
> >>
> >> Differential Revision: https://reviews.llvm.org/D51495
> >>
> >> Modified:
> >> llvm/trunk/lib/CodeGen/RegisterPressure.cpp
> >> llvm/trunk/test/CodeGen/X86/GlobalISel/gep.ll
> >> llvm/trunk/test/CodeGen/X86/divrem.ll
> >> llvm/trunk/test/CodeGen/X86/divrem8_ext.ll
> >>
> >> Modified: llvm/trunk/lib/CodeGen/RegisterPressure.cpp
> >> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/RegisterPressure.cpp?rev=342457&r1=342456&r2=342457&view=diff
> >> ==============================================================================
> >> --- llvm/trunk/lib/CodeGen/RegisterPressure.cpp (original)
> >> +++ llvm/trunk/lib/CodeGen/RegisterPressure.cpp Tue Sep 18 02:53:42 2018
> >> @@ -681,8 +681,7 @@ void PressureDiff::addPressureChange(uns
> >> PressureDiff::iterator J;
> >> for (J = std::next(I); J != E && J->isValid(); ++J, ++I)
> >> *I = *J;
> >> - if (J != E)
> >> - *I = *J;
> >> + *I = PressureChange();
> >> }
> >> }
> >> }
> >>
> >> Modified: llvm/trunk/test/CodeGen/X86/GlobalISel/gep.ll
> >> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/GlobalISel/gep.ll?rev=342457&r1=342456&r2=342457&view=diff
> >> ==============================================================================
> >> --- llvm/trunk/test/CodeGen/X86/GlobalISel/gep.ll (original)
> >> +++ llvm/trunk/test/CodeGen/X86/GlobalISel/gep.ll Tue Sep 18 02:53:42 2018
> >> @@ -6,12 +6,12 @@ define i32* @test_gep_i8(i32 *%arr, i8 %
> >> ; X64_GISEL-LABEL: test_gep_i8:
> >> ; X64_GISEL: # %bb.0:
> >> ; X64_GISEL-NEXT: # kill: def $esi killed $esi def $rsi
> >> -; X64_GISEL-NEXT: movq $4, %rax
> >> ; X64_GISEL-NEXT: movq $56, %rcx
> >> ; X64_GISEL-NEXT: # kill: def $cl killed $rcx
> >> ; X64_GISEL-NEXT: shlq %cl, %rsi
> >> ; X64_GISEL-NEXT: movq $56, %rcx
> >> ; X64_GISEL-NEXT: # kill: def $cl killed $rcx
> >> +; X64_GISEL-NEXT: movq $4, %rax
> >> ; X64_GISEL-NEXT: sarq %cl, %rsi
> >> ; X64_GISEL-NEXT: imulq %rax, %rsi
> >> ; X64_GISEL-NEXT: leaq (%rdi,%rsi), %rax
> >> @@ -46,12 +46,12 @@ define i32* @test_gep_i16(i32 *%arr, i16
> >> ; X64_GISEL-LABEL: test_gep_i16:
> >> ; X64_GISEL: # %bb.0:
> >> ; X64_GISEL-NEXT: # kill: def $esi killed $esi def $rsi
> >> -; X64_GISEL-NEXT: movq $4, %rax
> >> ; X64_GISEL-NEXT: movq $48, %rcx
> >> ; X64_GISEL-NEXT: # kill: def $cl killed $rcx
> >> ; X64_GISEL-NEXT: shlq %cl, %rsi
> >> ; X64_GISEL-NEXT: movq $48, %rcx
> >> ; X64_GISEL-NEXT: # kill: def $cl killed $rcx
> >> +; X64_GISEL-NEXT: movq $4, %rax
> >> ; X64_GISEL-NEXT: sarq %cl, %rsi
> >> ; X64_GISEL-NEXT: imulq %rax, %rsi
> >> ; X64_GISEL-NEXT: leaq (%rdi,%rsi), %rax
> >>
> >> Modified: llvm/trunk/test/CodeGen/X86/divrem.ll
> >> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/divrem.ll?rev=342457&r1=342456&r2=342457&view=diff
> >> ==============================================================================
> >> --- llvm/trunk/test/CodeGen/X86/divrem.ll (original)
> >> +++ llvm/trunk/test/CodeGen/X86/divrem.ll Tue Sep 18 02:53:42 2018
> >> @@ -57,11 +57,11 @@ define void @si32(i32 %x, i32 %y, i32* %
> >> ; X32-LABEL: si32:
> >> ; X32: # %bb.0:
> >> ; X32-NEXT: pushl %esi
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %esi
> >> ; X32-NEXT: movl {{[0-9]+}}(%esp), %eax
> >> ; X32-NEXT: cltd
> >> ; X32-NEXT: idivl {{[0-9]+}}(%esp)
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %esi
> >> ; X32-NEXT: movl %eax, (%esi)
> >> ; X32-NEXT: movl %edx, (%ecx)
> >> ; X32-NEXT: popl %esi
> >> @@ -87,11 +87,11 @@ define void @si16(i16 %x, i16 %y, i16* %
> >> ; X32-LABEL: si16:
> >> ; X32: # %bb.0:
> >> ; X32-NEXT: pushl %esi
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %esi
> >> ; X32-NEXT: movzwl {{[0-9]+}}(%esp), %eax
> >> ; X32-NEXT: cwtd
> >> ; X32-NEXT: idivw {{[0-9]+}}(%esp)
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %esi
> >> ; X32-NEXT: movw %ax, (%esi)
> >> ; X32-NEXT: movw %dx, (%ecx)
> >> ; X32-NEXT: popl %esi
> >> @@ -117,11 +117,11 @@ define void @si8(i8 %x, i8 %y, i8* %p, i
> >> ; X32-LABEL: si8:
> >> ; X32: # %bb.0:
> >> ; X32-NEXT: pushl %ebx
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %edx
> >> ; X32-NEXT: movb {{[0-9]+}}(%esp), %al
> >> ; X32-NEXT: cbtw
> >> ; X32-NEXT: idivb {{[0-9]+}}(%esp)
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %edx
> >> ; X32-NEXT: movsbl %ah, %ebx
> >> ; X32-NEXT: movb %al, (%edx)
> >> ; X32-NEXT: movb %bl, (%ecx)
> >> @@ -199,11 +199,11 @@ define void @ui32(i32 %x, i32 %y, i32* %
> >> ; X32-LABEL: ui32:
> >> ; X32: # %bb.0:
> >> ; X32-NEXT: pushl %esi
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %esi
> >> ; X32-NEXT: movl {{[0-9]+}}(%esp), %eax
> >> ; X32-NEXT: xorl %edx, %edx
> >> ; X32-NEXT: divl {{[0-9]+}}(%esp)
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %esi
> >> ; X32-NEXT: movl %eax, (%esi)
> >> ; X32-NEXT: movl %edx, (%ecx)
> >> ; X32-NEXT: popl %esi
> >> @@ -229,11 +229,11 @@ define void @ui16(i16 %x, i16 %y, i16* %
> >> ; X32-LABEL: ui16:
> >> ; X32: # %bb.0:
> >> ; X32-NEXT: pushl %esi
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %esi
> >> ; X32-NEXT: movzwl {{[0-9]+}}(%esp), %eax
> >> ; X32-NEXT: xorl %edx, %edx
> >> ; X32-NEXT: divw {{[0-9]+}}(%esp)
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %esi
> >> ; X32-NEXT: movw %ax, (%esi)
> >> ; X32-NEXT: movw %dx, (%ecx)
> >> ; X32-NEXT: popl %esi
> >> @@ -259,11 +259,11 @@ define void @ui8(i8 %x, i8 %y, i8* %p, i
> >> ; X32-LABEL: ui8:
> >> ; X32: # %bb.0:
> >> ; X32-NEXT: pushl %ebx
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> -; X32-NEXT: movl {{[0-9]+}}(%esp), %edx
> >> ; X32-NEXT: movzbl {{[0-9]+}}(%esp), %eax
> >> ; X32-NEXT: # kill: def $eax killed $eax def $ax
> >> ; X32-NEXT: divb {{[0-9]+}}(%esp)
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %ecx
> >> +; X32-NEXT: movl {{[0-9]+}}(%esp), %edx
> >> ; X32-NEXT: movzbl %ah, %ebx
> >> ; X32-NEXT: movb %al, (%edx)
> >> ; X32-NEXT: movb %bl, (%ecx)
> >>
> >> Modified: llvm/trunk/test/CodeGen/X86/divrem8_ext.ll
> >> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/divrem8_ext.ll?rev=342457&r1=342456&r2=342457&view=diff
> >> ==============================================================================
> >> --- llvm/trunk/test/CodeGen/X86/divrem8_ext.ll (original)
> >> +++ llvm/trunk/test/CodeGen/X86/divrem8_ext.ll Tue Sep 18 02:53:42 2018
> >> @@ -150,8 +150,8 @@ define i8 @test_srem_noext_ah(i8 %x, i8
> >> ; X32-LABEL: test_srem_noext_ah:
> >> ; X32: # %bb.0:
> >> ; X32-NEXT: movb {{[0-9]+}}(%esp), %al
> >> -; X32-NEXT: movb {{[0-9]+}}(%esp), %cl
> >> ; X32-NEXT: cbtw
> >> +; X32-NEXT: movb {{[0-9]+}}(%esp), %cl
> >> ; X32-NEXT: idivb %cl
> >> ; X32-NEXT: movsbl %ah, %eax
> >> ; X32-NEXT: addb %cl, %al
> >>
> >>
> >> _______________________________________________
> >> 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