[PATCH] D54547: PTH-- Remove feature entirely-

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 26 06:38:51 PST 2018


erichkeane marked 2 inline comments as done.
erichkeane added inline comments.


================
Comment at: include/clang/Lex/Preprocessor.h:396
+  /// This is an alias for CurLexer.
   PreprocessorLexer *CurPPLexer = nullptr;
 
----------------
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.  


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54547/new/

https://reviews.llvm.org/D54547





More information about the cfe-commits mailing list