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? Regards, Venkat.