[Lldb-commits] [lldb] [LLDB] Update DIL to pass current 'frame var' tests. (PR #145055)

[via lldb-commits]

cmtice wrote:

> If the test you found the anonymous namespace issue it is explicitly testing the handling of anonymous namespaces, then this is fine. However, if it's doing that as a side-effect of testing something else, then it may be nice to have an explicit test case for it. (Particularly as the anonymous namespace thing can appear in other positions as well (`ns1::(anonymous namespace)::foo`, `(anonymous namespace)::ns::(anonymous namespace)::foo`, etc.)

The test I found is lldb-api :: lang/cpp/namespace/TestNamespace.py, which does a basic test of '(anonymous namespace)' at the start of a variable name, but not embedded in the variable name.  I agree that we should test it in other positions as well. I will add that to one of the DIL tests.

https://github.com/llvm/llvm-project/pull/145055