vitalybuka updated this revision to Diff 134385.
vitalybuka added a comment.
Herald added a subscriber: hiraditya.
Don't use empty index file
https://reviews.llvm.org/D42995
Files:
clang/lib/CodeGen/BackendUtil.cpp
clang/test/CodeGen/Inputs/thinlto-distributed-backend-skip.bc
clang/test/C
vitalybuka planned changes to this revision.
vitalybuka added a comment.
https://reviews.llvm.org/D42995
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tejohnson added a comment.
In https://reviews.llvm.org/D42995#1000155, @vitalybuka wrote:
> In https://reviews.llvm.org/D42995#125, @tejohnson wrote:
>
> > > Empty ThinLTOIndexFile signals that we don't need this module during
> > > linking.
> >
> > Not the only case actually. We now also pa
vitalybuka added a comment.
In https://reviews.llvm.org/D42995#1000155, @vitalybuka wrote:
> In https://reviews.llvm.org/D42995#125, @tejohnson wrote:
>
> > > Empty ThinLTOIndexFile signals that we don't need this module during
> > > linking.
> >
> > Not the only case actually. We now also p
vitalybuka added a comment.
In https://reviews.llvm.org/D42995#125, @tejohnson wrote:
> > Empty ThinLTOIndexFile signals that we don't need this module during
> > linking.
>
> Not the only case actually. We now also pass an empty index file when we want
> to compile the bitcode down to obje
tejohnson added a comment.
> Empty ThinLTOIndexFile signals that we don't need this module during
> linking.
Not the only case actually. We now also pass an empty index file when we want
to compile the bitcode down to object without applying any LTO optimization
(there are a few cases where we
vitalybuka created this revision.
vitalybuka added reviewers: pcc, tejohnson.
Herald added subscribers: eraman, inglorion, mehdi_amini.
Empty ThinLTOIndexFile signals that we don't need this module during
linking. So we should not run ThinLTO backend even if it contains the
ThinLTO module. Backend
