maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

DataFlowSanitizer: Remove unreachable BBs so IR continues to verify 

under the args ABI.

Differential Revision: http://llvm-reviews.chandlerc.com/D1316

llvm-svn: 188113
This commit is contained in:
Peter Collingbourne 2013-08-09 21:42:53 +00:00
parent 4b03d98858
commit ae66d57bcf
2 changed files with 23 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
llvm/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp
View File

@ -59,6 +59,7 @@
#include "llvm/Pass.h" #include "llvm/Pass.h"
#include "llvm/Support/CommandLine.h" #include "llvm/Support/CommandLine.h"
#include "llvm/Transforms/Utils/BasicBlockUtils.h" #include "llvm/Transforms/Utils/BasicBlockUtils.h"
#include "llvm/Transforms/Utils/Local.h"
#include "llvm/Transforms/Utils/SpecialCaseList.h" #include "llvm/Transforms/Utils/SpecialCaseList.h"
#include <iterator> #include <iterator>
@ -402,6 +403,8 @@ bool DataFlowSanitizer::runOnModule(Module &M) {
if ((*i)->isDeclaration()) if ((*i)->isDeclaration())
continue; continue;
removeUnreachableBlocks(**i);
DFSanFunction DFSF(*this, *i); DFSanFunction DFSF(*this, *i);
// DFSanVisitor may create new basic blocks, which confuses df_iterator. // DFSanVisitor may create new basic blocks, which confuses df_iterator.

20
llvm/test/Instrumentation/DataFlowSanitizer/args-unreachable-bb.ll Normal file
View File

@ -0,0 +1,20 @@
; RUN: opt < %s -dfsan -verify -dfsan-args-abi -S | FileCheck %s
target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128-n8:16:32:64-S128"
define i8 @unreachable_bb() {
; CHECK: @unreachable_bb
; CHECK: ret { i8, i16 } { i8 1, i16 0 }
; CHECK-NOT: bb2:
; CHECK-NOT: bb3:
; CHECK-NOT: bb4:
ret i8 1
bb2:
ret i8 2
bb3:
br label %bb4
bb4:
br label %bb3
}