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

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

In D130229#3669754 <https://reviews.llvm.org/D130229#3669754>, @weiwang wrote:

> 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.

Actually we specifically don't want to do that (include the selected objects in --start-lib/--end-lib in the index output), we already know which files the linker chose to include from the archive groupings and we don't want to redo that logic which could theoretically come to a different conclusion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130229