[PATCH] D17900: [gold] Fix common symbols handling
Rafael EspĂndola via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 15 16:51:56 PDT 2016
On 15 March 2016 at 13:17, Evgeniy Stepanov <eugenis at google.com> wrote:
> eugenis added a comment.
>
> Something weird happened here. I've accidentally committed this as r263180 instead of http://reviews.llvm.org/D17360. I need to learn arc.
>
> The good news is the bots are happy.
> Do you want me to revert, or could we make this a post-commit review?
No need to revert. The patch looks reasonable. It is unfortunate that
the common symbol is so convoluted, but I think that is a consequence
of the plugin api.
Cheers,
Rafael
More information about the llvm-commits
mailing list