[PATCH] D60974: Clang IFSO driver action.

Jake Ehrlich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 29 17:49:59 PDT 2019


jakehehrlich added a comment.

With respect to group handeling. I think we should just check that the details of every symbol across all unmerged files that are being merged are consistent and then ignore duplicates.

For symbols that are not defined in sections that are a part of a comdat group this is just delaying the error to the linker. For symbols defined in a comdat group this is the only way we can be consistent with whatever choice the linker makes. In practice every comdat group is the same but if it were different the linker is allowed to just pick one arbitrarily. The only way to be consistent is to ensure that duplicates are identical. This however means we ignore duplicate definitions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60974





More information about the cfe-commits mailing list