Emit function alias to data as a function symbol.
authorEvgeniy Stepanov <eugeni.stepanov@gmail.com>
Fri, 4 Dec 2015 00:45:43 +0000 (00:45 +0000)
committerEvgeniy Stepanov <eugeni.stepanov@gmail.com>
Fri, 4 Dec 2015 00:45:43 +0000 (00:45 +0000)
CFI emits jump slots for indirect functions as a byte array
constant, and declares function-typed aliases to these constants.

This change fixes AsmPrinter to emit these aliases as function
symbols and not data symbols.

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

lib/CodeGen/AsmPrinter/AsmPrinter.cpp
test/CodeGen/Generic/function-alias.ll [new file with mode: 0644]

index 2cfea650872a5cb8f75532caa1c593540d1610e9..b8604240b5d9c33d7103a53a9cb2bdd8e259a07f 100644 (file)
@@ -1187,6 +1187,11 @@ bool AsmPrinter::doFinalization(Module &M) {
     else
       assert(Alias.hasLocalLinkage() && "Invalid alias linkage");
 
+    // Set the symbol type to function if the alias has a function type.
+    // This affects codegen when the aliasee is not a function.
+    if (Alias.getType()->getPointerElementType()->isFunctionTy())
+      OutStreamer->EmitSymbolAttribute(Name, MCSA_ELF_TypeFunction);
+
     EmitVisibility(Name, Alias.getVisibility());
 
     // Emit the directives as assignments aka .set:
diff --git a/test/CodeGen/Generic/function-alias.ll b/test/CodeGen/Generic/function-alias.ll
new file mode 100644 (file)
index 0000000..d68d75d
--- /dev/null
@@ -0,0 +1,12 @@
+; RUN: llc < %s | FileCheck %s
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
+
+; "data" constant
+@0 = private constant <{ i8, i8 }> <{i8 15, i8 11}>, section ".text"
+
+; function-typed alias
+@ud2 = alias void (), bitcast (<{ i8, i8 }>* @0 to void ()*)
+
+; Check that "ud2" is emitted as a function symbol.
+; CHECK: .type{{.*}}ud2,@function