Review Request 122160: Use buddy as fallback for headers without explicitly pinned TU.

Milian Wolff mail at milianw.de
Wed Apr 22 14:35:38 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122160/
-----------------------------------------------------------

(Updated April 22, 2015, 2:35 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDevelop.


Changes
-------

Submitted with commit 5b3106cd618d9d849787e7075c8142cfbca397a3 by Milian Wolff to branch master.


Repository: kdev-clang


Description
-------

In such cases, it's worthwile to use the source buddy as best-guess as
it allows us to get rid of the parse run for the "header TU" which
gets thrown away later on anyways.

Performance wise, this patch improves the index run on a 48 file
project using STL: It goes from ~17s down to ~14s, just b/c we safe
a few needless clang runs on "header TUs".

REVIEW: 122160


Diffs
-----

  CMakeLists.txt a28f0c1925bf20bfbde22e4dc0fd702960f4e3cd 
  clangparsejob.cpp d4c24d314598a17e043c4427d1781f6d7551b6a5 
  clangsupport.cpp f2c1a1ac7a0a366e698ae836954b397a95015ecc 
  documentfinderhelpers.h  
  documentfinderhelpers.cpp  
  duchain/CMakeLists.txt 8ed282b652361c45c668fa948917e075fed4f277 
  duchain/clangindex.cpp 25c7855e7bd67aa2a3be28f1ea803dd025e6a1ba 

Diff: https://git.reviewboard.kde.org/r/122160/diff/


Testing
-------


Thanks,

Milian Wolff

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150422/a75b0471/attachment.html>


More information about the KDevelop-devel mailing list