[PATCH] D47679: [lldb, lldb-mi] Enable lldb-mi -break-insert test on Windows.
Stella Stamenova via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Jun 2 15:51:16 PDT 2018
stella.stamenova added a comment.
You can actually follow the pattern in the other tests and do:
%cc -o %t %p/inputs/break-insert.c -g
%t``` then becomes the name of the output file and you can use that in our other commands. This is better than specifying a hardcoded name.
Having said that, I believe this won't fix the test. If you look at the comments of the review when I disabled the test, there is output of what happens when you give it the correct name - which is to run until the program exits *normally* rather than hitting a breakpoint as the test expects. After the program exits normally, lldb-mi just waits and the test will hang. Your other change will address the issue of the test hanging when it's given the incorrect file name, but I don't think it will fix this case.
Repository:
rL LLVM
https://reviews.llvm.org/D47679
More information about the llvm-commits
mailing list