]> granicus.if.org Git - llvm/commitdiff
CodeGen: Don't follow aliases when extracting type info.
authorPeter Collingbourne <peter@pcc.me.uk>
Fri, 2 Aug 2019 17:43:45 +0000 (17:43 +0000)
committerPeter Collingbourne <peter@pcc.me.uk>
Fri, 2 Aug 2019 17:43:45 +0000 (17:43 +0000)
This fixes a crash in the case where the type info object is an alias
pointing to a non-zero offset within a global or is otherwise unanalyzable
by the stripPointerCasts() function. Looking through the alias is not the
right thing to do anyway for similar reasons as D65118.

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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@367696 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/Analysis.cpp
test/CodeGen/X86/typeid-alias.ll [new file with mode: 0644]

index d158e70b86acad4e48099b2f93a45a90b637c783..e72938ba9bec438207b40d477688973d379b9a8c 100644 (file)
@@ -156,7 +156,7 @@ void llvm::computeValueLLTs(const DataLayout &DL, Type &Ty,
 
 /// ExtractTypeInfo - Returns the type info, possibly bitcast, encoded in V.
 GlobalValue *llvm::ExtractTypeInfo(Value *V) {
-  V = V->stripPointerCasts();
+  V = V->stripPointerCastsNoFollowAliases();
   GlobalValue *GV = dyn_cast<GlobalValue>(V);
   GlobalVariable *Var = dyn_cast<GlobalVariable>(V);
 
diff --git a/test/CodeGen/X86/typeid-alias.ll b/test/CodeGen/X86/typeid-alias.ll
new file mode 100644 (file)
index 0000000..885ab70
--- /dev/null
@@ -0,0 +1,18 @@
+; Test that we don't crash in the case where the type info object is an
+; alias pointing to a non-zero offset within a global.
+
+; RUN: llc < %s | FileCheck %s
+
+target triple = "x86_64-unknown-linux-gnu"
+
+%struct.exception = type { i8, i8, i32, i8*, i8*, i32, i8* }
+@g = global %struct.exception zeroinitializer
+@a = alias %struct.exception, %struct.exception* inttoptr (i64 add (i64 ptrtoint (%struct.exception* @g to i64), i64 1297036692682702848) to %struct.exception*)
+
+define i32 @typeid() {
+  ; CHECK: movl $1, %eax
+  %eh_typeid = tail call i32 @llvm.eh.typeid.for( i8* getelementptr (%struct.exception, %struct.exception* @a, i32 0, i32 0))
+  ret i32 %eh_typeid
+}
+
+declare i32 @llvm.eh.typeid.for(i8*)