zturner added a comment.

In https://reviews.llvm.org/D53662#1275238, @jingham wrote:

> I worry that your patch changes the behavior when you add the type_class 
> explicitly in the lookup (i.e. look up "struct Struct" not "Struct".  That 
> should work...
>
> Note, this doesn't currently work in type lookup:
>
>   (lldb) type lookup "struct Foo"
>   no type was found matching '"struct Foo"'
>


Ahh, nice catch.  Luckily it should be easy to add a couple of tests for those 
alongside the other ones I added, so I'll try that and upload a new version.


https://reviews.llvm.org/D53662



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to