[all-commits] [llvm/llvm-project] 4aee81: [lldb][NFC] Allow creating ClangExpressionDeclMap ...

Raphael Isemann via All-commits all-commits at lists.llvm.org
Tue Dec 17 05:05:22 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 4aee81c4f73359230e358108bc428e3b0cc59566
      https://github.com/llvm/llvm-project/commit/4aee81c4f73359230e358108bc428e3b0cc59566
  Author: Raphael Isemann <teemperor at gmail.com>
  Date:   2019-12-17 (Tue, 17 Dec 2019)

  Changed paths:
    M lldb/source/Plugins/ExpressionParser/Clang/ClangASTSource.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/ClangASTSource.h
    M lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.h
    M lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.cpp
    M lldb/source/Symbol/ClangASTContext.cpp
    M lldb/unittests/Expression/CMakeLists.txt
    A lldb/unittests/Expression/ClangExpressionDeclMapTest.cpp

  Log Message:
  -----------
  [lldb][NFC] Allow creating ClangExpressionDeclMap and ClangASTSource without a Target and add basic unit test

The ClangExpressionDeclMap should be testable from a unit test. This is currently
impossible as they have both dependencies on Target/ExecutionContext from their
constructor. This patch allows constructing these classes without an active Target
and adds the missing tests for running without a target that we can do at least
a basic lookup test without crashing.




More information about the All-commits mailing list