================
@@ -0,0 +1,25 @@
+#include <cstdio>
----------------
Teemperor wrote:

In general, tests should never include system headers unless needed. It mainly 
causes slow compilation times (this simple include adds a few hundred lines and 
opens 50 files, which can be slow on debug builds and some systems). It also 
adds a dependency on OS headers where some tests start failing because of OS 
header contents and that's not fun to debug.

You can remove this and simply add a `// break here` comment at the `return r;` 
instead of a `puts`.

https://github.com/llvm/llvm-project/pull/203984
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to