This is a testcase for https://bugs.llvm.org/show_bug.cgi?id=36777, which is
fixed by https://reviews.llvm.org/D44614.
Details
- Reviewers
grimar ruiu • espindola
Diff Detail
- Repository
- rLLD LLVM Linker
Event Timeline
Having such test case in LLD seems reasonable to me.
test/ELF/linkerscript/cxx11-tag-version.s | ||
---|---|---|
4 | Please use: | |
25 | I do not think you need to check anything except the names here. So I would shorten the checks: # CHECK: _Z14accepts_stringRKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE # CHECK: _Z14returns_stringB5cxx11v # CHECK: _Z6globalB5cxx11 | |
34 | And I would drop the comments, they do not seem to be useful. |
"cxx11-tag-version.s" is not an appropriate file name for this test case because this is not specific to C++11. I'd name "extern-cpp.s".
test/ELF/linkerscript/cxx11-tag-version.s | ||
---|---|---|
3 | I think you can remove "VER1". | |
8 | Remove extraneous space characters between "CHECK:" and "Name:" | |
12–25 | You don't need to repeat the same test case three times. Please use only one symbol. |
I think you can remove "VER1".