[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