[not for merging] clang-cl wip

[nico]

Hi, this isn't done, but has a few bits that help (none of them mergable):

1. Making _LoadCompdb() take the wanted path and return data for just that instead of building a whole dict and then using only one entry reduced whodis compdb parse time from minutes to < 1s for me in a chrome/win cross build
2. The /FO stuff is probably how it's going to have to look
3. /FA /Z7 probably too, but in 32-bit builds it runs into some clang-internal bugs (https://llvm.org/PR36676)
4. The codeview debug info is pretty different from the DWARF one in the asm file, and the codeview info parsing code isn't done yet.

So just FYI in case you (or someone else) wants to look at this.

[sgraham]

Cool! :)

On Mon, Mar 12, 2018 at 7:49 AM, Nico Weber ***@***.***> wrote: Hi, this isn't done, but has a few bits that help (none of them mergable): 1. Making _LoadCompdb() take the wanted path and return data for just that instead of building a whole dict and then using only one entry reduced whodis compdb parse time from minutes to < 1s for me in a chrome/win cross build

Yikes! I haven't seen >1s before, but I guess the computer I'm using is too fast/caching too well.

1. The /FO stuff is probably how it's going to have to look 2. /FA /Z7 probably too, but in 32-bit builds it runs into some clang-internal bugs (https://llvm.org/PR36676) 3. The codeview debug info is pretty different from the DWARF one in the asm file, and the codeview info parsing code isn't done yet. So just FYI in case you (or someone else) wants to look at this.

I hadn't tried at all with non-Linux/Mac clang, but looks pretty reasonable. Let me know when you're happy for a merge.

…

------------------------------

You can view, comment on, or merge this pull request online at: #4 Commit Summary - wip for making whodis work for clang-cl compiles File Changes - *M* plugin/whodis.vim <https://github.com/sgraham/whodis/pull/4/files#diff-0> (51) Patch Links: - https://github.com/sgraham/whodis/pull/4.patch - https://github.com/sgraham/whodis/pull/4.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#4>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAHHkMT3PRRyWMJn_am0rZaMPpNCmv6ks5tdorkgaJpZM4SmxsT> .

[asankah]

The _LoadCompdb() stuff would be great to have! Without it I'm also looking at > 1m wait times.