diff options
| author | Rafael Espindola <rafael.espindola@gmail.com> | 2014-07-30 22:51:54 +0000 |
|---|---|---|
| committer | Rafael Espindola <rafael.espindola@gmail.com> | 2014-07-30 22:51:54 +0000 |
| commit | 464fe024c53180caaf3a06a45409b04905c27c58 (patch) | |
| tree | b55d811630df41bf7e9050ee44064157455bdff9 /llvm/test/Transforms/InstCombine/objsize-address-space.ll | |
| parent | c5fe19d06265fcde822cd549d3c90d27c11e320e (diff) | |
| download | bcm5719-llvm-464fe024c53180caaf3a06a45409b04905c27c58.tar.gz bcm5719-llvm-464fe024c53180caaf3a06a45409b04905c27c58.zip | |
Use "weak alias" instead of "alias weak"
Before this patch we had
@a = weak global ...
but
@b = alias weak ...
The patch changes aliases to look more like global variables.
Looking at some really old code suggests that the reason was that the old
bison based parser had a reduction for alias linkages and another one for
global variable linkages. Putting the alias first avoided the reduce/reduce
conflict.
The days of the old .ll parser are long gone. The new one parses just "linkage"
and a later check is responsible for deciding if a linkage is valid in a
given context.
llvm-svn: 214355
Diffstat (limited to 'llvm/test/Transforms/InstCombine/objsize-address-space.ll')
| -rw-r--r-- | llvm/test/Transforms/InstCombine/objsize-address-space.ll | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/test/Transforms/InstCombine/objsize-address-space.ll b/llvm/test/Transforms/InstCombine/objsize-address-space.ll index 9cb68842396..a971c913bd4 100644 --- a/llvm/test/Transforms/InstCombine/objsize-address-space.ll +++ b/llvm/test/Transforms/InstCombine/objsize-address-space.ll @@ -32,7 +32,7 @@ define i16 @foo_as3_i16() nounwind { ret i16 %1 } -@a_alias = alias weak [60 x i8] addrspace(3)* @a_as3 +@a_alias = weak alias [60 x i8] addrspace(3)* @a_as3 define i32 @foo_alias() nounwind { %1 = call i32 @llvm.objectsize.i32.p3i8(i8 addrspace(3)* getelementptr inbounds ([60 x i8] addrspace(3)* @a_alias, i32 0, i32 0), i1 false) ret i32 %1 |
