karies added a comment.
Herald added a project: All.
Similar to the concern raised at
https://github.com/llvm/llvm-project/commit/50fcf7285eeb001d751eadac5d001a0e216fd701
we have received user reports about this patch: with `-Ino-access-permissions
-Iall-good`, clang will throw an error (EACCES
This revision was automatically updated to reflect the committed changes.
Closed by commit rL368322: clang: Diag running out of file handles while
looking for files (authored by nico, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to commit:
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65956/new/
https://reviews.llvm.org/D65956
___
cfe-commits mailing list
cfe-commits@lists.l
thakis created this revision.
thakis added a reviewer: rnk.
clang would only print "file not found" when it's unable to find a
header file. If the reason for that is a file handle leak, that's not a
very useful error message. For errors that aren't in a small whitelist
("file not found", "file i
