[PATCH] D21890: [ELF] - Do not error out when version declaration not found when building executable.

[George Rimar via llvm-commits]

grimar added a comment.

In http://reviews.llvm.org/D21890#472210, @ruiu wrote:

> That logic makes sense, but this is also satisfies our need, no?
>
>   if (HaveNoVersionScript) {
>     Config->SymbolVersions.push_back(elf::Version(Version));
>     return Default ? I : (I | VERSYM_HIDDEN);
>   }


Technically yes. I just think than when linking DSO with versioned
symbols it is assumed that you have the script. If you do not have it then
it is more probably a mistake, like you forget to specify one. This code
can hide such possible error I think.
I would be more strict here and error out such cases.


http://reviews.llvm.org/D21890