search for: d16440

...e of not pulling the > DICompileUnit when no functions referencing it is in the split module). > Then each thread would do redundant work processing this type hierarchy > (and other debug info). > > For ThinLTO, Teresa is taking care (review waiting here: > http://reviews.llvm.org/D16440 ) to try to import as little as possible, > and turn type definition into declaration when possible. > Right - I don't think we'd ever need to import a definition - just rely on the fact that we will produce a type definition somewhere in the output (this may present problems for LLD...

Sorry if this has been discussed already... I am trying to understand whether there is an underlying reason why llvm::CloneModule takes argument function ShouldCloneDefinition to discriminate on objects being cloned, but blindly copies all named metadata...? In my case this results in debug nodes that are creating uses/relocations for global objects that were _not_ copied per

...> Related work > ============ > > Teresa has some post-processing in-place/in-review to avoid importing > metadata unnecessarily, but IIUC: it won't address (a) and (b), only > (c) (maybe I'm wrong?); and it only helps -flto=thin, not other > lazy-loaders. > That is D16440. It reduces the metadata imported into the dest module (not sure whether that falls into (b) or just (c)). It could actually help full LTO as well since I also added support for not linking in unneeded DISubprogram for full LTO at the same time as ThinLTO in r256003. But right now the changes in t...

...elated work > ============ > > Teresa has some post-processing in-place/in-review to avoid importing > metadata unnecessarily, but IIUC: it won't address (a) and (b), only > (c) (maybe I'm wrong?); and it only helps -flto=thin, not other > lazy-loaders. > > That is D16440. It reduces the metadata imported into the dest module (not sure whether that falls into (b) or just (c)). > > It could actually help full LTO as well since I also added support for not linking in unneeded DISubprogram for full LTO at the same time as ThinLTO in r256003. But right now the ch...

...; > > > Teresa has some post-processing in-place/in-review to avoid importing > > metadata unnecessarily, but IIUC: it won't address (a) and (b), only > > (c) (maybe I'm wrong?); and it only helps -flto=thin, not other > > lazy-loaders. > > > > That is D16440. It reduces the metadata imported into the dest module > (not sure whether that falls into (b) or just (c)). > > > > It could actually help full LTO as well since I also added support for > not linking in unneeded DISubprogram for full LTO at the same time as > ThinLTO in r2560...

On Fri, Feb 5, 2016 at 5:51 PM Mehdi Amini via llvm-dev < llvm-dev at lists.llvm.org> wrote: > > > On Feb 5, 2016, at 5:40 PM, Peter Collingbourne <peter at pcc.me.uk> wrote: > > > > On Fri, Feb 05, 2016 at 04:58:45PM -0800, Mehdi Amini wrote: > >> > >>> On Feb 5, 2016, at 3:17 PM, Peter Collingbourne via llvm-dev < > llvm-dev at

...; > > > Teresa has some post-processing in-place/in-review to avoid importing > > metadata unnecessarily, but IIUC: it won't address (a) and (b), only > > (c) (maybe I'm wrong?); and it only helps -flto=thin, not other > > lazy-loaders. > > > > That is D16440. It reduces the metadata imported into the dest module (not sure whether that falls into (b) or just (c)). > > > > It could actually help full LTO as well since I also added support for not linking in unneeded DISubprogram for full LTO at the same time as ThinLTO in r256003. But right n...

...ndicated that he and Adrian were going to be looking at this as well shortly, I wanted to send out a summary of what David Blaikie and I discussed, what I have right now, and the issues I am hitting along with possible solutions. There was a related older patch where I did some of the same things (D16440: [ThinLTO] Link in only necessary DICompileUnit operands), but that was made obsolete by a number of changes, such as the reversal of GlobalVariable/DIGlobalVariable edges, and a number of debug metadata changes done by Duncan (and Adrian I think?). So I started from scratch this time. Here's...

I have some ideas to allow the BitcodeReader to lazy-load debug info metadata, and wanted to air this on llvm-dev before getting too deep into the code. Motivation ========== Based on some analysis Mehdi ran (ping him for details), there are three (related) compile-time bottlenecks we're seeing with `-flto=thin -g`: a) Reading the large number of Metadata bitcode records in the global

...oing to be looking at this as well shortly, I wanted to send out a summary > of what David Blaikie and I discussed, what I have right now, and the > issues I am hitting along with possible solutions. > > > > There was a related older patch where I did some of the same things > (D16440: [ThinLTO] Link in only necessary DICompileUnit operands), but that > was made obsolete by a number of changes, such as the reversal of > GlobalVariable/DIGlobalVariable edges, and a number of debug metadata > changes done by Duncan (and Adrian I think?). So I started from scratch > thi...

...oing to be looking at this as well shortly, I wanted to send out a summary > of what David Blaikie and I discussed, what I have right now, and the > issues I am hitting along with possible solutions. > > > > There was a related older patch where I did some of the same things > (D16440: [ThinLTO] Link in only necessary DICompileUnit operands), but that > was made obsolete by a number of changes, such as the reversal of > GlobalVariable/DIGlobalVariable edges, and a number of debug metadata > changes done by Duncan (and Adrian I think?). So I started from scratch > thi...