diff options
| author | Hal Finkel <hfinkel@anl.gov> | 2015-02-18 03:12:28 +0000 |
|---|---|---|
| committer | Hal Finkel <hfinkel@anl.gov> | 2015-02-18 03:12:28 +0000 |
| commit | 4393559621a3b2784086ba4d30be635426d2da31 (patch) | |
| tree | 4458f87bf55f553d3279f1406b3e80d509043cd3 /llvm/test | |
| parent | 5927e7b6815d19c5381053d948925b621d17bdfc (diff) | |
| download | bcm5719-llvm-4393559621a3b2784086ba4d30be635426d2da31.tar.gz bcm5719-llvm-4393559621a3b2784086ba4d30be635426d2da31.zip | |
[BDCE] Don't forget uses of root instructions seen before the instruction itself
When visiting the initial list of "root" instructions (those which must always
be alive), for those that are integer-valued (such as invokes returning an
integer), we mark their bits as (initially) all dead (we might, obviously, find
uses of those bits later, but all bits are assumed dead until proven
otherwise). Don't do so, however, if we're already seen a use of those bits by
another root instruction (such as a store).
Fixes a miscompile of the sanitizer unit tests on x86_64.
Also, add a debug line for visiting the root instructions, and remove a debug
line which tried to print instructions being removed (printing dead
instructions is dangerous, and can sometimes crash).
llvm-svn: 229618
Diffstat (limited to 'llvm/test')
| -rw-r--r-- | llvm/test/Transforms/BDCE/order.ll | 37 |
1 files changed, 37 insertions, 0 deletions
diff --git a/llvm/test/Transforms/BDCE/order.ll b/llvm/test/Transforms/BDCE/order.ll new file mode 100644 index 00000000000..301f4476ab5 --- /dev/null +++ b/llvm/test/Transforms/BDCE/order.ll @@ -0,0 +1,37 @@ +; RUN: opt -bdce -S < %s | FileCheck %s +target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128" +target triple = "x86_64-unknown-linux-gnu" + +declare i32 @__gxx_personality_v0(...) + +define fastcc void @_ZN11__sanitizerL12TestRegistryEPNS_14ThreadRegistryEb() #0 { +entry: + br i1 undef, label %if.else, label %entry.if.end_crit_edge + +if.else: + ret void + +invoke.cont70: + store i32 %call71, i32* undef, align 4 + br label %if.else + +; CHECK-LABEL: @_ZN11__sanitizerL12TestRegistryEPNS_14ThreadRegistryEb +; CHECK: store i32 %call71 + +lpad65.loopexit.split-lp.loopexit.split-lp: + br label %if.else + +lpad65.loopexit.split-lp.loopexit.split-lp.loopexit: + %lpad.loopexit1121 = landingpad { i8*, i32 } personality i8* bitcast (i32 (...)* @__gxx_personality_v0 to i8*) + cleanup + br label %lpad65.loopexit.split-lp.loopexit.split-lp + +entry.if.end_crit_edge: + %call71 = invoke i32 @_ZN11__sanitizer14ThreadRegistry12CreateThreadEmbjPv() + to label %invoke.cont70 unwind label %lpad65.loopexit.split-lp.loopexit.split-lp.loopexit +} + +declare i32 @_ZN11__sanitizer14ThreadRegistry12CreateThreadEmbjPv() + +attributes #0 = { uwtable } + |
