[all-commits] [llvm/llvm-project] 32baf5: [lldb][expr] Propagate ClangDynamicCheckerFunction...

Stefan Gränitz via All-commits all-commits at lists.llvm.org
Tue Mar 21 11:06:28 PDT 2023

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 32baf5c1c29b6b2f282354c9f5919865bc1ff958
  Author: Stefan Gränitz <stefan.graenitz at gmail.com>
  Date:   2023-03-21 (Tue, 21 Mar 2023)

  Changed paths:
    M lldb/include/lldb/Expression/DynamicCheckerFunctions.h
    M lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/IRDynamicChecks.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/IRDynamicChecks.h

  Log Message:
  [lldb][expr] Propagate ClangDynamicCheckerFunctions::Install() errors to caller

I came accross this, because a lot of regression tests were saying:
(lldb) p argc
error: expression failed to parse:
error: couldn't install checkers, unknown error

With this change, error messages provide more detail:
(lldb) p argc
error: expression failed to parse:
error: couldn't install checkers:
error: Couldn't lookup symbols:

I didn't find a case where `Diagnostics()` is not empty. Also it looks like this isn't covered in any test (yet).

Reviewed By: bulbazord, Michael137

Differential Revision: https://reviews.llvm.org/D146541

More information about the All-commits mailing list