vsapsai added a comment.
Thanks for the prompt review.
Repository:
rC Clang
https://reviews.llvm.org/D45603
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
This revision was automatically updated to reflect the committed changes.
Closed by commit rC330041: Fix evaluation of `__has_include_next` during
-frewrite-includes. (authored by vsapsai, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D45603?vs=142299=142438#toc
bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.
lg
https://reviews.llvm.org/D45603
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
vsapsai added a comment.
To simplify reviewing additions to the test, here is the corresponding output
# 23
"/Users/vsapsai/Projects/llvm/llvm-project/clang/test/Frontend/rewrite-includes.c"
2
#if 0 /* expanded by -frewrite-includes */
#include "rewrite-includes9.h"
#endif /* expanded
vsapsai created this revision.
vsapsai added a reviewer: bkramer.
Herald added a subscriber: jkorous-apple.
`__has_include_next` requires correct DirectoryLookup for being
evaluated correctly. We were using Preprocessor::GetCurDirLookup() but
we were calling it after the preprocessor finished its
