https://github.com/jasonmolenda approved this pull request.

This looks good to me.  Do you think we take the frame variable test in 
API/functionalities/location-list-lookup/TestLocationListLookup.py and add it 
to your test?  They're testing the same thing, and getting there the same way 
so I'm not sure there's value to having the two separate tests for the frame 
variable codepath and the expr codepath.

https://github.com/llvm/llvm-project/pull/74772
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to