[PATCH] D46110: [IR] Do not assume that function pointers are aligned

[Mikhail Maltsev via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL331025: [IR] Do not assume that function pointers are aligned (authored by miyuki, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D46110?vs=144147&id=144299#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D46110

Files:
  llvm/trunk/lib/IR/Value.cpp
  llvm/trunk/test/Analysis/ValueTracking/func-ptr-lsb.ll


Index: llvm/trunk/test/Analysis/ValueTracking/func-ptr-lsb.ll
===================================================================
--- llvm/trunk/test/Analysis/ValueTracking/func-ptr-lsb.ll
+++ llvm/trunk/test/Analysis/ValueTracking/func-ptr-lsb.ll
@@ -0,0 +1,18 @@
+; RUN: opt -instcombine -S < %s | FileCheck %s
+
+target datalayout = "e-p:32:32-n32-S64"
+
+; CHECK-LABEL: @foo_ptr
+; CHECK: and
+define i32 @foo_ptr() {
+entry:
+  ; Even though the address of @foo is aligned, we cannot assume that the
+  ; pointer has the same alignment. This is not true for e.g. ARM targets
+  ; which store ARM/Thumb state in the LSB
+  ret i32 and (i32 ptrtoint (void ()* @foo to i32), i32 -4)
+}
+
+define internal void @foo() align 16 {
+entry:
+  ret void
+}
Index: llvm/trunk/lib/IR/Value.cpp
===================================================================
--- llvm/trunk/lib/IR/Value.cpp
+++ llvm/trunk/lib/IR/Value.cpp
@@ -685,6 +685,10 @@
 
   unsigned Align = 0;
   if (auto *GO = dyn_cast<GlobalObject>(this)) {
+    // Don't make any assumptions about function pointer alignment. Some
+    // targets use the LSBs to store additional information.
+    if (isa<Function>(GO))
+      return 0;
     Align = GO->getAlignment();
     if (Align == 0) {
       if (auto *GVar = dyn_cast<GlobalVariable>(GO)) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46110.144299.patch
Type: text/x-patch
Size: 1309 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180427/a08fef96/attachment.bin>