[PATCH] D63974: [ELF] Only allow the binding of SharedSymbol to change for the first undef ref

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 2 04:34:57 PDT 2019


MaskRay marked an inline comment as done.
MaskRay added a comment.

In D63974#1566354 <https://reviews.llvm.org/D63974#1566354>, @ruiu wrote:

> Maybe it's good to have a static_assert to ensure that the symbol size is 80 bytes.


The static_assert change may be a bit risky (32-bit 64-bit ELF, Windows, etc). Let me do it in another change.


Repository:
  rLLD LLVM Linker

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63974/new/

https://reviews.llvm.org/D63974





More information about the llvm-commits mailing list