Change the linkage of these global values to 'internal'.
authorBill Wendling <isanbard@gmail.com>
Tue, 6 Aug 2013 22:52:42 +0000 (22:52 +0000)
committerBill Wendling <isanbard@gmail.com>
Tue, 6 Aug 2013 22:52:42 +0000 (22:52 +0000)
commit55a1a590bf0cadf88dfbef2aab6948ffec35c1c3
treec8d7a3fb9a7689831f8851c4675f675417e9dbd8
parent47cfec02842f885b46ea0d3c812793e660691640
Change the linkage of these global values to 'internal'.

The globals being generated here were given the 'private' linkage type. However,
this caused them to end up in different sections with the wrong prefix. E.g.,
they would be in the __TEXT,__const section with an 'L' prefix instead of an 'l'
(lowercase ell) prefix.

The problem is that the linker will eat a literal label with 'L'. If a weak
symbol is then placed into the __TEXT,__const section near that literal, then it
cannot distinguish between the literal and the weak symbol.

Part of the problems here was introduced because the address sanitizer converted
some C strings into constant initializers with trailing nuls. (Thus putting them
in the __const section with the wrong prefix.) The others were variables that
the address sanitizer created but simply had the wrong linkage type.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@187827 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Transforms/Instrumentation/AddressSanitizer.cpp
test/Instrumentation/AddressSanitizer/do-not-instrument-internal-globals.ll