[PATCH] D54547: PTH-- Remove feature entirely-
Bruno Ricci via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 26 06:43:07 PST 2018
riccibruno added inline comments.
================
Comment at: include/clang/Lex/Preprocessor.h:396
+ /// This is an alias for CurLexer.
PreprocessorLexer *CurPPLexer = nullptr;
----------------
erichkeane wrote:
> riccibruno wrote:
> > Would it make sense to remove this alias now that it
> > will always refer to `CurLexer` ?
> There seems to be some logic around this that I'm not totally sure about dealing with IsFileLexer.
Sounds fine to me. Someone more familiar with this can
always come back later and rework this.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D54547/new/
https://reviews.llvm.org/D54547
More information about the cfe-commits
mailing list