aaron.ballman added a comment.

> Sema.h is huge.

Understatement of the year. :-D

Thank you for working on this, I think it's a nice little cleanup. Just some 
minor nits.



================
Comment at: clang/include/clang/Sema/EnterExpressionEvaluationContext.h:1
+//===--- Sema.h - Semantic Analysis & AST Building --------------*- C++ 
-*-===//
+//
----------------
Comments here aren't in sync with reality.


================
Comment at: clang/include/clang/Sema/EnterExpressionEvaluationContext.h:70
+#endif
\ No newline at end of file

----------------
Please add the newline to the end of the file.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149796

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to