jimingham wrote: The change this is repairing did cause a failure - the one described in:
https://github.com/llvm/llvm-project/issues/169913 I have tests for breakpoint serialization and deserialization (which is where this broke) in TestBreakpointSerialization.py but those tests never check that the deserialized breakpoints have the expected number of locations - which is why this wasn't caught. Should be pretty easy to add that extra check, and then there would be at least one test of a failure that the original patch caused in llvm-project. https://github.com/llvm/llvm-project/pull/168797 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
