> On 13 Feb 2015, at 11:14, Richard Earnshaw <richard.earns...@foss.arm.com> > wrote: > >> Is this ok? > > The repetitive nature of all these new cpus being added looks rather > wooden. I think it would be better to merge them into one change block, > that lists all the cpus and their internal names, then mentions once at > the end that these names can be used as arguments to -mcpu and -mtune.
I wholeheartedly agree with Richard on this. >> + <li>Support for the Applied Micro X-Gene processor is now available >> through >> + the <code>-mcpu=xgene1</code> and <code>-mtune=xgene1</code> options. >> + </li> Please change this to “X-Gene 1 processor”, as there will be a separate option for “X-Gene 2” in the future (which will switch to slightly different tuning parameters). Best, Philipp.