From d490f7b9d452ade9d25ccfb5dfef3fb25d3a6eea Mon Sep 17 00:00:00 2001 From: David Green Date: Thu, 13 Aug 2020 22:40:36 +0100 Subject: [PATCH] Revert "[ARM] Fix IT block generation after Thumb2SizeReduce with -Oz" This reverts commit 18279a54b5d3382874924d6a3c7775b7e22598dc as it is causing some chromium android test problems. --- lib/Target/ARM/ARMBaseInstrInfo.cpp | 7 ------ test/CodeGen/Thumb2/constant-hoisting.ll | 27 ++++++++++++------------ 2 files changed, 14 insertions(+), 20 deletions(-) diff --git a/lib/Target/ARM/ARMBaseInstrInfo.cpp b/lib/Target/ARM/ARMBaseInstrInfo.cpp index 8be1d10eba0..ab7aaa597b4 100644 --- a/lib/Target/ARM/ARMBaseInstrInfo.cpp +++ b/lib/Target/ARM/ARMBaseInstrInfo.cpp @@ -587,13 +587,6 @@ bool ARMBaseInstrInfo::DefinesPredicate( const MachineOperand &MO = MI.getOperand(i); if ((MO.isRegMask() && MO.clobbersPhysReg(ARM::CPSR)) || (MO.isReg() && MO.isDef() && MO.getReg() == ARM::CPSR)) { - - // Filter out T1 instructions that have a dead CPSR, - // allowing IT blocks to be generated containing T1 instructions - const MCInstrDesc &MCID = MI.getDesc(); - if (MCID.TSFlags & ARMII::ThumbArithFlagSetting && MO.isDead()) - continue; - Pred.push_back(MO); Found = true; } diff --git a/test/CodeGen/Thumb2/constant-hoisting.ll b/test/CodeGen/Thumb2/constant-hoisting.ll index a106900dc3e..5c8f934ce61 100644 --- a/test/CodeGen/Thumb2/constant-hoisting.ll +++ b/test/CodeGen/Thumb2/constant-hoisting.ll @@ -37,25 +37,26 @@ define i32 @test_values(i32 %a, i32 %b) minsize optsize { ; CHECK-V7M: mov r2, r0 ; CHECK-V7M-NEXT: ldr r0, .LCPI0_0 ; CHECK-V7M-NEXT: cmp r2, #50 -; CHECK-V7M-NEXT: beq .LBB0_3 +; CHECK-V7M-NEXT: beq .LBB0_5 ; CHECK-V7M-NEXT: cmp r2, #1 -; CHECK-V7M-NEXT: ittt eq -; CHECK-V7M-NEXT: addeq r0, r1 -; CHECK-V7M-NEXT: addeq r0, #1 -; CHECK-V7M-NEXT: bxeq lr +; CHECK-V7M-NEXT: beq .LBB0_7 ; CHECK-V7M-NEXT: cmp r2, #30 -; CHECK-V7M-NEXT: ittt eq -; CHECK-V7M-NEXT: addeq r0, r1 -; CHECK-V7M-NEXT: addeq r0, #2 -; CHECK-V7M-NEXT: bxeq lr -; CHECK-V7M-NEXT: cbnz r2, .LBB0_4 -; CHECK-V7M-NEXT: .LBB0_2: +; CHECK-V7M-NEXT: beq .LBB0_8 +; CHECK-V7M-NEXT: cbnz r2, .LBB0_6 ; CHECK-V7M-NEXT: add r0, r1 ; CHECK-V7M-NEXT: bx lr -; CHECK-V7M-NEXT: .LBB0_3: +; CHECK-V7M-NEXT: .LBB0_5: ; CHECK-V7M-NEXT: add r0, r1 ; CHECK-V7M-NEXT: adds r0, #4 -; CHECK-V7M-NEXT: .LBB0_4: +; CHECK-V7M-NEXT: .LBB0_6: +; CHECK-V7M-NEXT: bx lr +; CHECK-V7M-NEXT: .LBB0_7: +; CHECK-V7M-NEXT: add r0, r1 +; CHECK-V7M-NEXT: adds r0, #1 +; CHECK-V7M-NEXT: bx lr +; CHECK-V7M-NEXT: .LBB0_8: +; CHECK-V7M-NEXT: add r0, r1 +; CHECK-V7M-NEXT: adds r0, #2 ; CHECK-V7M-NEXT: bx lr ; CHECK-V7M-NEXT: .p2align 2 ; CHECK-V7M-NEXT: .LCPI0_0: