[all-commits] [llvm/llvm-project] 11ecd4: [LLDB] Update DIL to pass current 'frame var' test...

[cmtice via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 11ecd4742b74697b8331d0051e533302e4d17c85
      https://github.com/llvm/llvm-project/commit/11ecd4742b74697b8331d0051e533302e4d17c85
  Author: cmtice <cmtice at google.com>
  Date:   2025-07-01 (Tue, 01 Jul 2025)

  Changed paths:
    M lldb/source/Target/StackFrame.cpp
    M lldb/source/ValueObject/DILEval.cpp
    M lldb/source/ValueObject/DILParser.cpp
    M lldb/test/API/commands/frame/var-dil/basics/MemberOf/TestFrameVarDILMemberOf.py
    M lldb/test/API/commands/frame/var-dil/basics/MemberOfAnonymousMember/TestFrameVarDILMemberOfAnonymousMember.py

  Log Message:
  -----------
  [LLDB] Update DIL to pass current 'frame var' tests. (#145055)

As a preliminary to making DIL the default implementation for
 'frame var', ran check-lldb forcing 'frame var' to always use DIL, 
and discovered a few failing tests. This fixes most of them. The only
remaining failing test is TestDAP_evaluate.py, which now passes
a test case that the test says should fail (still investigating this).

Changes in this PR:
- Sets correct VariableSP, as well as returning ValueObjectSP (needed
for several watchpoint tests).
- Updates error messages, when looking up members, to match what the
rest of LLDB expects. Also update appropriate DIL tests to expect the
updated error messages.
- Updates DIL parser to look for and accept "(anonymous namespace)::" at
the front of a variable name.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications