[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