Skip to content

Commit

Permalink
[DCE] Add DebugCounter support
Browse files Browse the repository at this point in the history
Patch by Zhizhou Yang!

Differential Revision: https://reviews.llvm.org/D50092

llvm-svn: 342170
  • Loading branch information
gburgessiv committed Sep 13, 2018
1 parent 008982d commit 7a8dc3d
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 0 deletions.
8 changes: 8 additions & 0 deletions llvm/lib/Transforms/Scalar/DCE.cpp
Expand Up @@ -24,13 +24,16 @@
#include "llvm/IR/InstIterator.h"
#include "llvm/IR/Instruction.h"
#include "llvm/Pass.h"
#include "llvm/Support/DebugCounter.h"
#include "llvm/Transforms/Scalar.h"
using namespace llvm;

#define DEBUG_TYPE "dce"

STATISTIC(DIEEliminated, "Number of insts removed by DIE pass");
STATISTIC(DCEEliminated, "Number of insts removed");
DEBUG_COUNTER(DCECounter, "dce-transform",
"Controls which instructions are eliminated");

namespace {
//===--------------------------------------------------------------------===//
Expand All @@ -50,6 +53,8 @@ namespace {
for (BasicBlock::iterator DI = BB.begin(); DI != BB.end(); ) {
Instruction *Inst = &*DI++;
if (isInstructionTriviallyDead(Inst, TLI)) {
if (!DebugCounter::shouldExecute(DCECounter))
continue;
salvageDebugInfo(*Inst);
Inst->eraseFromParent();
Changed = true;
Expand Down Expand Up @@ -77,6 +82,9 @@ static bool DCEInstruction(Instruction *I,
SmallSetVector<Instruction *, 16> &WorkList,
const TargetLibraryInfo *TLI) {
if (isInstructionTriviallyDead(I, TLI)) {
if (!DebugCounter::shouldExecute(DCECounter))
return false;

salvageDebugInfo(*I);

// Null out all of the instruction's operands to see if any operand becomes
Expand Down
20 changes: 20 additions & 0 deletions llvm/test/Other/debugcounter-dce.ll
@@ -0,0 +1,20 @@
; REQUIRES: asserts
; RUN: opt -dce -S -debug-counter=dce-transform-skip=1,dce-transform-count=2 < %s | FileCheck %s
;; Test that, with debug counters on, we will skip the first DCE opportunity, perform next 2,
;; and ignore all the others left.

; CHECK-LABEL: @test
; CHECK-NEXT: %add1 = add i32 1, 2
; CHECK-NEXT: %sub1 = sub i32 %add1, 1
; CHECK-NEXT: %add2 = add i32 1, 2
; CHECK-NEXT: %add3 = add i32 1, 2
; CHECK-NEXT: ret void
define void @test() {
%add1 = add i32 1, 2
%sub1 = sub i32 %add1, 1
%add2 = add i32 1, 2
%sub2 = sub i32 %add2, 1
%add3 = add i32 1, 2
%sub3 = sub i32 %add3, 1
ret void
}

0 comments on commit 7a8dc3d

Please sign in to comment.