[PATCH] D48946: [Preamble] Check system dependencies in preamble too

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 9 01:52:48 PDT 2018


ilya-biryukov added a comment.

In https://reviews.llvm.org/D48946#1152764, @sammccall wrote:

> This seems like it might be a nontrivial performance hit (it's going to result in `stat`ing all these files, right?).
>  Agreed it's important for correctness, it's possible someone wants the performance/correctness tradeoff though.


I do agree that it's (probably) useless to stat the STL in /usr/lib for most of the users, maybe in some other cases too. Happy to add an option if that turns out to be a problem.


Repository:
  rC Clang

https://reviews.llvm.org/D48946





More information about the cfe-commits mailing list