[Lldb-commits] [lldb] [lldb][test] Introduce build_and_run test utility (PR #194386)

[Raphael Isemann via lldb-commits]

Teemperor wrote:

* Added ability to set comment/file name (thanks Ismail!)
* Also returning the result of lldbutil.run_to_source_breakpoint which several tests need to do some checks.

The plan here isn't to make all tests just call this function, but all tests that do this simple task (build -> create target -> breakpoint->launch) should call this function and then follow the existing convention (file named `main.*` and a comment called `// break here`). I still think there are a few cases where a custom name/comment makes sense, so making it an argument sounds good to me.

FWIW: I'm not saying this is a perfect convention, but it's better than a quarter of the tests having completely different conventions than the rest. Ideally, we should match `break here.` to avoid C++ style comments in C tests and avoid matching random `break here` words in a test, but that's out of scope for this PR.

https://github.com/llvm/llvm-project/pull/194386