[PATCH] D54187: Add debuginfo-tests that use cdb on Windows

[Adrian Prantl via Phabricator via cfe-commits]

aprantl added a comment.

Did anyone take the time to look at dexter and whether it would fit the bill here? It would be great to avoid reimplementing its functionality just to have something that then only works on Windows.

My position is that for the kind of very basic end-to-end testing that we do in this repository (set breakpoints+print variables), it should be possible to have one wrapper that supports multiple debuggers. I'm also fine with adding platform-specific tests into a subdirectory, but that should be the exception, not the rule. What I would like to avoid is growing a large parallel body of basic tests that only work on one platform.

It sounded like dexter would be the missing glue layer that supports windows debuggers and unix debuggers alike. If that is feasible I would prefer to extend dexter to support WinDbg and migrating debuginfo-tests to use dexter over adding new code and more complexity to debuginfo-tests.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54187/new/

https://reviews.llvm.org/D54187