[PATCH] D80406: [LLD][ThinLTO] A switch to allow compilation of only one module.

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 28 10:23:39 PDT 2020


hoyFB added a comment.

In D80406#2060565 <https://reviews.llvm.org/D80406#2060565>, @tejohnson wrote:

> In D80406#2057639 <https://reviews.llvm.org/D80406#2057639>, @tejohnson wrote:
>
> > Can you beef up the testing? I.e. ensure that the save temps output files for just the specified module get created. Also please add a test to make sure this interacts well with distributed thinlto build options, i.e. -thinlto-index-only (e.g. that it creates the expected output files just for the specified module).
>
>
> Not sure if you saw this comment from my earlier reply - please add additional testing as suggested above.


It's added. E.g, the `not ls single1.o2` line checks only two intermediate objects are generated, one is the dummy `ld-temp.o` and the other has the symbol specified in the command line. The `IDX` checks are for thinlto index testing. Do you want me to add more? Thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80406





More information about the llvm-commits mailing list