[PATCH] D30667: GVNHoist: handle basic blocks with UnreachableInst

Sebastian Pop via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 7 18:06:50 PST 2017


This revision was automatically updated to reflect the committed changes.
Closed by commit rL297254: Handle UnreachableInst in isGuaranteedToTransferExecutionToSuccessor (authored by spop).

Changed prior to commit:
  https://reviews.llvm.org/D30667?vs=90750&id=90976#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D30667

Files:
  llvm/trunk/lib/Analysis/ValueTracking.cpp
  llvm/trunk/test/Transforms/GVNHoist/hoist-very-busy.ll


Index: llvm/trunk/test/Transforms/GVNHoist/hoist-very-busy.ll
===================================================================
--- llvm/trunk/test/Transforms/GVNHoist/hoist-very-busy.ll
+++ llvm/trunk/test/Transforms/GVNHoist/hoist-very-busy.ll
@@ -0,0 +1,34 @@
+; RUN: opt -S -gvn-hoist < %s | FileCheck %s
+
+%struct.__jmp_buf_tag = type { [8 x i64], i32 }
+
+; Check that hoisting only happens when the expression is very busy.
+; CHECK: store
+; CHECK: store
+
+ at test_exit_buf = global %struct.__jmp_buf_tag zeroinitializer
+ at G = global i32 0
+
+define void @test_command(i32 %c1) {
+entry:
+  switch i32 %c1, label %exit [
+    i32 0, label %sw0
+    i32 1, label %sw1
+  ]
+
+sw0:
+  store i32 1, i32* @G
+  br label %exit
+
+sw1:
+  store i32 1, i32* @G
+  br label %exit
+
+exit:
+  call void @longjmp(%struct.__jmp_buf_tag* @test_exit_buf, i32 1) #0
+  unreachable
+}
+
+declare void @longjmp(%struct.__jmp_buf_tag*, i32) #0
+
+attributes #0 = { noreturn nounwind }
Index: llvm/trunk/lib/Analysis/ValueTracking.cpp
===================================================================
--- llvm/trunk/lib/Analysis/ValueTracking.cpp
+++ llvm/trunk/lib/Analysis/ValueTracking.cpp
@@ -3781,6 +3781,8 @@
     return false;
   if (isa<ReturnInst>(I))
     return false;
+  if (isa<UnreachableInst>(I))
+    return false;
 
   // Calls can throw, or contain an infinite loop, or kill the process.
   if (auto CS = ImmutableCallSite(I)) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30667.90976.patch
Type: text/x-patch
Size: 1441 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170308/af86d051/attachment.bin>


More information about the llvm-commits mailing list