- User Since
- May 26 2014, 12:49 PM (273 w, 3 d)
Mon, Aug 19
Ugh nevermind, you can't because of the binary files. Fineeeeee.
I know it's super lame to request this, but would one of you mind submitting on my behalf? I don't have SVN installed and it seems like git llvm push still requires it (unless there is a more modern workflow that doesn't require SVN anymore). If I find myself submitting more and more patches then I'll bite the bullet.
Tue, Aug 13
No objections from me on the wording of the layering portion
Jul 16 2019
Apr 29 2019
Apr 19 2019
I just built against trunk with VS 2019 and ran into this exact issue. llvm-tblgen had no command line options. Are more fixes still in the pipeline or was this thought to be sufficient?
Apr 12 2019
Apr 11 2019
Apr 10 2019
Apr 9 2019
Apr 8 2019
Apr 5 2019
Apr 3 2019
Also remove LLDBTriggerable.py
Add the scheduler back but remove the android builder only.
Apr 2 2019
Apr 1 2019
Can't you just write a function that, every time you call it, traces the symbol back to its original compile unit (you can get this from the PdbCompilandSymId), get the CompileUnit item, and ask it for its language? The part that seems unnecessary is the cache.
Is the nature of the PDB different enough that it warrants an entirely new implementation? There's a efw places where we parse decorated / undecorated names, but in general I would imagine most of the stuff is language agnostic no?
Do you have a single PDB with multiple source file languages in it? This seems like it is going to consume a ton of memory if there's one of these for each SymUID
Mar 29 2019
Nice, does this actually have any performance impact? Since the arrays are smaller and have better cache locality.
Mar 28 2019
Mar 26 2019
Mar 21 2019
Mar 20 2019
If we ever wanted more aggressive pruning of dead type information, it probably makes sense to do that as a post-processing step where we re-write the PDB, something like an llvm-pdbutil gc subcommand.
Updated based on suggestions, including removal of the m_dwarf_data member variable which holds the contents of the __DWARF segment on MachO.
Mar 19 2019
I went ahead and just removed the const get version entirely, while changing the other function name to getOrLoad as you suggested. I have another patch while I'll upload after this one that converts all of the rest of the functions (except for the virtual ones required for DWO support), and the const version of the function was literally never needed, except in one place that was itself dead code. So, in the spirit of YAGNI, it's gone until it demonstrates a need.
This LGTM, but let's give it a day to see if anyone else chimes in with comments.