[PATCH] D130229: [ELF] Add --thinlto-index= and --remapping-file=

Wei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 21 12:30:42 PDT 2022


weiwang added a comment.

In D130229#3669642 <https://reviews.llvm.org/D130229#3669642>, @tejohnson wrote:

>> If there is a case, we probably have to give --remapping-file additional semantics that a specified file ignores the surrounding `--start-lib`. This makes  --remapping-file less generic, but we can probably add the third column to indicate whether --start-lib should be ignored.
>
> Can we just have the file generated by thinlto-index-only= also include native objects?

I agree that making `thinlto-index-only=` to produce the entire input list, both prebuilt native and bitcode-compiled native, is a less disruptive change to still keep the current cmdline options while making it more general. I think we can probably extend the the index file output to also include `--start-lib/--end-lib` grouping to that the final link can just use it as is.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130229



More information about the llvm-commits mailing list