[libcxx-commits] [PATCH] D155521: Run picolibc tests with qemu

[Michael Platings via Phabricator via libcxx-commits]

michaelplatings added inline comments.


================
Comment at: libcxx/utils/run-qemu.sh:19
+
+"${QEMU_EXECUTABLE}" \
+    -chardev stdio,mux=on,id=stdio0 \
----------------
arichardson wrote:
> arichardson wrote:
> > I wonder if this should instead be a full executor such as qemu_system.py (similar to ssh.py) that accepts all the same flags and just requires something like `--qemu=qemu-system-arm --cpu=.. --machine=...`
> Done as part of https://github.com/llvm/llvm-project/pull/68643, let me know if that works for you?
I'm not sure whether the Python script is an improvement but I'd go with whatever gets these patches landed sooner. I think that to be used in place of `run.py` the Python script would need to support the full interface of `run.py` i.e. `--codesign_identity` et al.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155521