summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRafael Espindola <rafael.espindola@gmail.com>2014-05-06 16:48:58 +0000
committerRafael Espindola <rafael.espindola@gmail.com>2014-05-06 16:48:58 +0000
commitb8894488415a81bb6bdaf9851acacc93a4e7cfe2 (patch)
tree2492746ffe3883ef48a18427846a8634262594c2
parentbfc3f301b6a5390b990c4abb05d134ffe8bb59bb (diff)
downloadllvm-b8894488415a81bb6bdaf9851acacc93a4e7cfe2.tar.gz
llvm-b8894488415a81bb6bdaf9851acacc93a4e7cfe2.tar.bz2
llvm-b8894488415a81bb6bdaf9851acacc93a4e7cfe2.tar.xz
Special case aliases in GlobalValue::getAlignment.
An alias has the address of what it points to, so it also has the same alignment. This allows a few optimizations to see past aliases for free. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@208103 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--include/llvm/IR/GlobalValue.h4
-rw-r--r--lib/IR/Globals.cpp7
-rw-r--r--lib/IR/Verifier.cpp1
-rw-r--r--test/CodeGen/AArch64/global-alignment.ll14
4 files changed, 22 insertions, 4 deletions
diff --git a/include/llvm/IR/GlobalValue.h b/include/llvm/IR/GlobalValue.h
index 7c4dd0796c..5d0e3791e8 100644
--- a/include/llvm/IR/GlobalValue.h
+++ b/include/llvm/IR/GlobalValue.h
@@ -81,9 +81,7 @@ public:
removeDeadConstantUsers(); // remove any dead constants using this.
}
- unsigned getAlignment() const {
- return (1u << Alignment) >> 1;
- }
+ unsigned getAlignment() const;
void setAlignment(unsigned Align);
bool hasUnnamedAddr() const { return UnnamedAddr; }
diff --git a/lib/IR/Globals.cpp b/lib/IR/Globals.cpp
index 76baa6202a..2265e4c2b7 100644
--- a/lib/IR/Globals.cpp
+++ b/lib/IR/Globals.cpp
@@ -63,6 +63,13 @@ void GlobalValue::copyAttributesFrom(const GlobalValue *Src) {
setDLLStorageClass(Src->getDLLStorageClass());
}
+unsigned GlobalValue::getAlignment() const {
+ if (auto *GA = dyn_cast<GlobalAlias>(this))
+ return GA->getAliasedGlobal()->getAlignment();
+
+ return (1u << Alignment) >> 1;
+}
+
void GlobalValue::setAlignment(unsigned Align) {
assert((!isa<GlobalAlias>(this)) &&
"GlobalAlias should not have an alignment!");
diff --git a/lib/IR/Verifier.cpp b/lib/IR/Verifier.cpp
index 0bc5c509eb..bc378aed16 100644
--- a/lib/IR/Verifier.cpp
+++ b/lib/IR/Verifier.cpp
@@ -477,7 +477,6 @@ void Verifier::visitGlobalAlias(const GlobalAlias &GA) {
"Alias and aliasee types should match!", &GA);
Assert1(!GA.hasUnnamedAddr(), "Alias cannot have unnamed_addr!", &GA);
Assert1(!GA.hasSection(), "Alias cannot have a section!", &GA);
- Assert1(!GA.getAlignment(), "Alias connot have an alignment", &GA);
const Constant *Aliasee = GA.getAliasee();
const GlobalValue *GV = dyn_cast<GlobalValue>(Aliasee);
diff --git a/test/CodeGen/AArch64/global-alignment.ll b/test/CodeGen/AArch64/global-alignment.ll
index 24e6618e98..36b74e5a57 100644
--- a/test/CodeGen/AArch64/global-alignment.ll
+++ b/test/CodeGen/AArch64/global-alignment.ll
@@ -4,6 +4,7 @@
@var32 = global [3 x i32] zeroinitializer
@var64 = global [3 x i64] zeroinitializer
@var32_align64 = global [3 x i32] zeroinitializer, align 8
+@alias = alias [3 x i32]* @var32_align64
define i64 @test_align32() {
; CHECK-LABEL: test_align32:
@@ -47,6 +48,19 @@ define i64 @test_var32_align64() {
ret i64 %val
}
+define i64 @test_var32_alias() {
+; CHECK-LABEL: test_var32_alias:
+ %addr = bitcast [3 x i32]* @alias to i64*
+
+ ; Test that we can find the alignment for aliases.
+ %val = load i64* %addr
+; CHECK: adrp x[[HIBITS:[0-9]+]], alias
+; CHECK-NOT: add x[[HIBITS]]
+; CHECK: ldr x0, [x[[HIBITS]], {{#?}}:lo12:alias]
+
+ ret i64 %val
+}
+
@yet_another_var = external global {i32, i32}
define i64 @test_yet_another_var() {