Don't force symbols to be globals in .thumb_set.
authorRafael Espindola <rafael.espindola@gmail.com>
Thu, 1 May 2014 12:45:43 +0000 (12:45 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Thu, 1 May 2014 12:45:43 +0000 (12:45 +0000)
commit1b68a68c0409cb30dcdfbac1d04a6d5541dedd36
tree655446f47ab118a9dcff647ed85442d340799995
parent3152576ea4768187f545efbffa1b542ab48eb9bc
Don't force symbols to be globals in .thumb_set.

We currently force symbols to be globals in .thumb_set. The intent
seems to be that given

.thumb_set foo, bar

we emit an undefined symbol to bar if it is never defined. The side
effect is that we mark bar as global, even if it is defined, which gas
does not.

Producing an undefined reference to bar is a general difference from MC and gas.
For example, given

a = b

gas will produce an undefined reference to b, MC will not. I would be surprised
if any code depends on this, but it it does, we should fix the general
difference, not special case .thumb_set.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@207757 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp
test/MC/ARM/thumb_set.s