[PATCH] D42160: Fix llvm_unreachable being tripped in ExecutionEngine

Brad Moody via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 00:10:25 PST 2018


bmoody created this revision.
bmoody added a reviewer: lhames.
Herald added a subscriber: llvm-commits.

Add handling for GlobalAliases in ExecutionEngine::getConstantValue.


Repository:
  rL LLVM

https://reviews.llvm.org/D42160

Files:
  lib/ExecutionEngine/ExecutionEngine.cpp
  test/ExecutionEngine/Interpreter/alias.ll


Index: test/ExecutionEngine/Interpreter/alias.ll
===================================================================
--- /dev/null
+++ test/ExecutionEngine/Interpreter/alias.ll
@@ -0,0 +1,14 @@
+; RUN: %lli -force-interpreter %s
+
+define i32 @func() {
+entry:
+  ret i32 0
+}
+
+ at alias = alias i32 (), i32 ()* @func
+
+define i32 @main() {
+entry:
+  %call = call i32 @alias()
+  ret i32 %call
+}
Index: lib/ExecutionEngine/ExecutionEngine.cpp
===================================================================
--- lib/ExecutionEngine/ExecutionEngine.cpp
+++ lib/ExecutionEngine/ExecutionEngine.cpp
@@ -903,6 +903,9 @@
     Result.IntVal = cast<ConstantInt>(C)->getValue();
     break;
   case Type::PointerTyID:
+    while (auto *A = dyn_cast<GlobalAlias>(C)) {
+      C = A->getAliasee();
+    }
     if (isa<ConstantPointerNull>(C))
       Result.PointerVal = nullptr;
     else if (const Function *F = dyn_cast<Function>(C))


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42160.130099.patch
Type: text/x-patch
Size: 931 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180117/fa29414e/attachment-0001.bin>


More information about the llvm-commits mailing list