[Lldb-commits] [PATCH] D82064: [ARM64] Add QEMU testing environment setup guide for SVE testing

[Muhammad Omair Javaid via Phabricator via lldb-commits]

omjavaid added a comment.

In D82064#2223991 <https://reviews.llvm.org/D82064#2223991>, @JDevlieghere wrote:

> I didn't look at the scripts in detail but I have a few general (high level) comments:
>
> - I also feel this should go under `lldb/scripts`.
> - Can we put the documentations as RST under `lldb/docs` so it's rendered on the website? We already have bunch of developer resources there. That will certainly help with discovery.
> - LLVM generally encourages scripts to be written in Python. While it doesn't look like this will work out of the box on Windows, I can imagine someone might be interested in that down the line. I also think the bar is lower for new contributors to write Python than bash. I'm kind of on the fence here because there's a reasonable balance between just running commands and control flow, but then I've been scarred by the `build-script-impl` disaster in Swift so I lean towards better safe than sorry.

These scripts fairly optional and are more like developer help for someone who wants to give a quick test to any of LLVM tools on some target in absence of hardware and I agree someone may wanna use them on windows and quite likely I might be that someone in very near future. But I ll hold moving to python for now. I ll write an rst doc and arrange the scripts under lldb/scripts as suggested.


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

https://reviews.llvm.org/D82064