On Fri, Jul 10, 2015 at 9:40 AM, Kumar, Venkataramanan <venkataramanan.ku...@amd.com> wrote: > Hi Richard, > >> -----Original Message----- >> From: Richard Biener [mailto:richard.guent...@gmail.com] >> Sent: Thursday, July 09, 2015 8:03 PM >> To: Kumar, Venkataramanan >> Cc: Gerald Pfeifer (ger...@pfeifer.com); gcc-patches@gcc.gnu.org >> Subject: Re: [Patch wwwdocs] gcc-5/changes.html : Document AMD >> monitorx and mwaitx >> >> On Thu, Jul 9, 2015 at 4:28 PM, Kumar, Venkataramanan >> <venkataramanan.ku...@amd.com> wrote: >> > Hi Gerald, >> > >> > This patch documents about AMD instructions "mwaitx" and "monitorx" in >> GCC- 5 changes.html. >> > Please let me know if this ok to commit? >> > >> > Index: htdocs/gcc-5/changes.html >> > >> ========================================================== >> ========= >> > RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v >> > retrieving revision 1.121 >> > diff -r1.121 changes.html >> > 810a811,819 >> >> <li>Support for new AMD instructions <code>monitorx</code> and >> >> <code>mwaitx</code> has been added. This includes new intrinsic >> >> and built-in support. It is enabled through option <code>- >> mmwaitx</code>. >> >> The instructions <code>monitorx</code> and <code>mwaitx</code> >> >> implement the same functionality as the old <code>monitor</code> >> >> and <code>mwait</code> instructions. In addition >> <code>mwaitx</code> >> >> adds a configurable timer. The timer value is received as third >> >> argument and stored in register <code>%ebx</code>. GCC 5.2 is the >> >> first release to support these instructions.</li> >> > >> > I will send another patch for trunk as well. >> >> Please add it to a subsection containing GCC 5.2 changes, see older releases >> for examples. >> > > I updated the patch as per your comments. The format followed as seen in > GCC 4.3 release changes. > > Index: htdocs/gcc-5/changes.html > =================================================================== > RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v > retrieving revision 1.121 > diff -r1.121 changes.html > 969a970,985 >> >> <h2><a name="5.2">GCC 5.2</a></h2> >> >> <h2>Target Specific Changes</h2> >> >> <h3>IA-32/x86-64</h3> >> <ul> >> <li>Support for new AMD instructions <code>monitorx</code> and >> <code>mwaitx</code> has been added. This includes new intrinsic >> and built-in support. It is enabled through option >> <code>-mmwaitx</code>. >> The instructions <code>monitorx</code> and <code>mwaitx</code> >> implement the same functionality as the old <code>monitor</code> >> and <code>mwait</code> instructions. In addition <code>mwaitx</code> >> adds a configurable timer. The timer value is received as third >> argument and stored in register <code>%ebx</code>.</li> >> </ul> > > Please let me know if this is ok to commit?
Yes, this looks good > > Regards, > Venkat. > >