teemperor added inline comments.
================ Comment at: lldb/include/lldb/Symbol/TypeSystemClang.h:38 +#include "Plugins/ExpressionParser/Clang/ClangPersistentVariables.h" + ---------------- I would also just let this be part of the normal include list without the empty line (or wherever clang format decides to place this). ================ Comment at: lldb/source/Plugins/Language/CPlusPlus/BlockPointer.cpp:21 +#include "Plugins/ExpressionParser/Clang/ClangPersistentVariables.h" + ---------------- I would just keep this in the include list above and let clang-format do its thing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72946/new/ https://reviews.llvm.org/D72946 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
