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/Feature/aliases.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/Feature/aliases.ll')
| -rw-r--r-- | llvm/test/Feature/aliases.ll | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/test/Feature/aliases.ll b/llvm/test/Feature/aliases.ll index ad1d1b08901..c11fc476425 100644 --- a/llvm/test/Feature/aliases.ll +++ b/llvm/test/Feature/aliases.ll @@ -21,10 +21,10 @@ define i32 @foo_f() { ret i32 0 } -@bar_f = alias weak_odr %FunTy* @foo_f +@bar_f = weak_odr alias %FunTy* @foo_f @bar_ff = alias i32()* @bar_f -@bar_i = alias internal i32* @bar +@bar_i = internal alias i32* @bar @A = alias bitcast (i32* @bar to i64*) |
