<div dir="ltr">Hi,<div><br><div class="gmail_extra">I have a two days travel, so this reply is late.<br><br><div class="gmail_quote">2014-08-12 10:56 GMT+08:00 Chris Johns <span dir="ltr"><<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On 11/08/2014 12:24 am, Peng Fan wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
1. Fix getopt_long usage in rtl host. some shorthand options are not<br>
hanlded correctly, this patch fixes it.<br>
</blockquote>
<br></div>
Thanks for cleaning this up.<div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
2. Add a new option '--mach-flags'/'-m' to rtems-ld. This optarg of this<br>
option will be passed to xx-rtemsxx-gcc, it will be used the search lib<br>
dirs. Detailed msg is in the commit log of the patch.<br>
</blockquote>
<br></div>
I wonder if we need the explicit -march and -mcpu options and this or should we remove them and add a more general option that can include these flags. When I added the -march etc I thought this was all that was needed and that is proving to be a little naive.<br>
</blockquote><div>If -march and -mcpu are only passed to gcc to let gcc search the libs, I think we can add a more generic option. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
What do you think ?</blockquote><div><br></div><div>I think we can extract the 'machine' related flags from rtems bsp and build a table in rtl-host like the following:</div><div>struct bsp_flag {</div><div>char* bsp_name;</div>
<div>char* flags;</div><div>} ;<br></div><div>Here machine related flags in gcc is at <a href="https://gcc.gnu.org/onlinedocs/gcc-4.8.3/gcc/">https://gcc.gnu.org/onlinedocs/gcc-4.8.3/gcc/</a> chapter 3.17.</div><div>struct bsp_flag bsp_flags[RTEMS_BSP_NUMS];</div>
<div>alloc space</div><div>bsp_flags[0].bsp_name = bsp name from rtems source code</div><div>bsp_flags[0].flags = machine flags from rtems source code corresponding to the bsp</div><div>bsp_flags[1]</div><div>bsp_flags[2]<br>
</div><div>......</div><div>Using the machine flags, xxx_rtemsxx_gcc can search the related libs first, if not found, then search the common libs, because the machine related lib path is in the first.</div><div><br></div>
<div>Just my thought, the code above is not good. Hmm. using String, new and class in c++</div><div><br></div><div>I think we may pass a madantory bsp name to rtl-host, such as "--bsp xxxxxxx" , xxxxxxx means the bsp name</div>
<div><br></div><div>Regards,</div><div>Peng.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class=""><font color="#888888"><br>

<br>
Chris<br>
</font></span></blockquote></div><br></div></div></div>