[PATCH] D71161: [ELF] Update st_size when merging a common symbol with a shared symbol

[Peter Smith via Phabricator via llvm-commits]

peter.smith added a comment.

Reading through the PR I think that this is the right thing to. I think that there is a possibility of narrowing down the assumption that a Shared definition is common, by checking its address against the loadable segment containing BSS. I don't think it is worth doing that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71161