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

Zachary Turner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 6 16:17:07 PST 2018


zturner added a comment.

I think the only way to realistically make this work for all platforms would be to separate the source file from the input/output.  The source file would be the test case, and if you wanted to support a different debugger you would need to supply a different input/output file..

That said, as Reid mentioned, we've been talking about getting this going for years, so at this point, and there's a steep initial overhead in making something that abstracts over multiple debuggers.  So at least until we have something that works on Windows with some interesting tests, I think we shouldn't try to create the abstraction just yet.


https://reviews.llvm.org/D54187





More information about the cfe-commits mailing list