[PATCH] D63781: [ELF] Error on archive with missing index

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 26 11:10:06 PDT 2019


sbc100 added a comment.

This change only effects archives without an index.    If you previously created an archive with both bitcode *and* ELF files *and* you didn't create and index, the previous behaviour was to completely ignore the archive.   This seems bad to me.  The new behaviour is to give an error.   We could downgrade it to a warning but both GNU ld and wasm-ld give a hard error in this case.

Are you saying that ignoring these hybrid archives is a behavior that you want to keep?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63781





More information about the llvm-commits mailing list