D17858: clang: Also detect Clang builtin dirs at runtime on Unix

[René J.V. Bertin]

rjvbb updated this revision to Diff 49080.
rjvbb marked an inline comment as done.
rjvbb added a comment.


  Updated as discussed in my previous comment. I've also followed flherne's suggestion to move the check for the reference file file into `ClangHelpers::clangBuiltinIncludePath()`. This does require some changes elsewhere because it can now fail (return an empty string).

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D17858?vs=48367&id=49080

REVISION DETAIL
  https://phabricator.kde.org/D17858

AFFECTED FILES
  plugins/clang/clangsupport.cpp
  plugins/clang/duchain/clanghelpers.cpp
  plugins/clang/duchain/clanghelpers.h
  plugins/clang/duchain/parsesession.cpp

To: rjvbb, #kdevelop, flherne, mwolff
Cc: arrowd, mwolff, flherne, kdevelop-devel, glebaccon, hase, antismap, iodelay, geetamc, Pilzschaf, akshaydeo, surgenight
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20190109/6d0ef58d/attachment.html>