[Lldb-commits] [PATCH] D59040: Move ExpressionSourceCode.cpp -> ClangExpressionSourceCode.cpp

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Mar 6 12:19:52 PST 2019


jingham created this revision.
jingham added a reviewer: shafik.
Herald added subscribers: lldb-commits, jdoerfert, teemperor, abidh, jfb, aprantl, srhines.
Herald added a project: LLDB.

ExpressionSourceCode.cpp is in source/Expressions - which should indicate that it is a generic part of the expression parser.  In fact, almost everything it currently does is very Clang specific.  This change factors out the clang specific bits into ClangExpressionSourceCode.cpp.  It is an NFC change, but it will help to reduce conflicts in supporting other Expression Parsers (e.g. swift).

I am leaving ExpressionSourceCode.h in place even though at present it doesn't have much content.  In the swift.org repository we have support for emitting debug information for compiled expressions - which requires saving out the expression text, and that is handled by this class.  That's a sometimes handy feature.  This change will make it easier to merge in that facility, though I'll do that as a separate commit.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D59040

Files:
  include/lldb/Expression/ExpressionSourceCode.h
  lldb.xcodeproj/project.pbxproj
  source/Expression/ExpressionSourceCode.cpp
  source/Expression/LLVMUserExpression.cpp
  source/Expression/UserExpression.cpp
  source/Expression/UtilityFunction.cpp
  source/Plugins/ExpressionParser/Clang/ClangExpressionSourceCode.cpp
  source/Plugins/ExpressionParser/Clang/ClangExpressionSourceCode.h
  source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp
  source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.cpp





More information about the lldb-commits mailing list