[Lldb-commits] [PATCH] D16334: Fix TestSignedTypes.py by removing a bogus step-over

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Wed Jan 20 12:21:54 PST 2016


zturner added a comment.

In http://reviews.llvm.org/D16334#331420, @tberghammer wrote:

> It is true that every CL can be tested but a lot of change is going in to address a specific edge case generated by a specific compiler in a strange situation. To create a reliable test from it we have to commit in a compiled binary with the strange/incorrect debug info and then it will be a platform and architecture specific test what is also very hard to debug because you most likely can't recompile it with your own compiler. I am not sure we want to go down in this road.


You can test cases like this easily with unit tests and dependency injection.  Just make a mock interface that returns the exact values you need when queried for certain symbols or whatever.


http://reviews.llvm.org/D16334





More information about the lldb-commits mailing list