On 15/04/2015 17:08, Liviu Ionescu wrote: > >> On 15 Apr 2015, at 15:49, Leon Alrae <leon.al...@imgtec.com> wrote: >> >> ... If there aren't any objections for the proposed solution, then let's >> just cook up the patch. > > the changes related to moving --semihosting-cmdline to the end of the line > are in: > > git://git.code.sf.net/p/gnuarmeclipse/qemu gnuarmeclipse-qemu > > the gnuarmeclipse-dev branch, commit 8010d2 > > although I did not group the semihosting related variables in a structure, > that code may still be a good starting point (especially since it is already > functional in my fork). > > I suggest you compare the gnuarmeclipse-dev branch with master and get the > semihosting related patches. > > if you need any help, please don't hesitate to ask. > > alternatively I can create the patch, but the changes must also touch non-arm > code, and I have no experience there.
Since the new option and your ARM semi-hosting improvements already work for you I believe it would be much easier if you sent the patches first rather than us figuring out the changes from your branch. Then we could go from there reviewing / refining / testing etc. Thanks, Leon