[Lldb-commits] [PATCH] D54454: Be more permissive in what we consider a variable name.

Zachary Turner via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Nov 12 17:06:14 PST 2018

zturner added a comment.

BTW, I will have to see if it's possible to write a test for this.  Even when I compiled and built a program with DWARF on Linux, the `target variable Pi<double>` example didn't "just work" for me, because `FindGlobalVariables` wasn't returning the variable.  So I think this part actually needs to be fixed in the DWARF plugin, which I'm not equipped to fix.  I can try something that is not a variable template, such as the `Foo<int>::StaticMember` example, but if that also doesn't work, then there might not be a good way to write a general purpose test for this until this is fixed in the DWARF plugin.

I can definitely add a test in the native pdb plugin though, and technically that runs everywhere (although it would only test `target variable` and not `frame variable`, maybe that's ok though?).


More information about the lldb-commits mailing list