[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