RE: [Patch wwwdocs] gcc-5/changes.html : Document AMD monitorx and mwaitx

2015-07-10 Thread Kumar, Venkataramanan
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
liSupport for new AMD instructions codemonitorx/code and
codemwaitx/code has been added. This includes new intrinsic
and built-in support. It is enabled through option code-
 mmwaitx/code.
The instructions codemonitorx/code and codemwaitx/code
implement the same functionality as the old codemonitor/code
and codemwait/code instructions. In addition
 codemwaitx/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
 
 h2a name=5.2GCC 5.2/a/h2
 
 h2Target Specific Changes/h2
 
 h3IA-32/x86-64/h3
   ul
 liSupport for new AMD instructions codemonitorx/code and
   codemwaitx/code has been added. This includes new intrinsic
   and built-in support. It is enabled through option 
 code-mmwaitx/code.
   The instructions codemonitorx/code and codemwaitx/code
   implement the same functionality as the old codemonitor/code
   and codemwait/code instructions. In addition codemwaitx/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.

 


Re: [Patch wwwdocs] gcc-5/changes.html : Document AMD monitorx and mwaitx

2015-07-10 Thread Richard Biener
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
liSupport for new AMD instructions codemonitorx/code and
codemwaitx/code has been added. This includes new intrinsic
and built-in support. It is enabled through option code-
 mmwaitx/code.
The instructions codemonitorx/code and codemwaitx/code
implement the same functionality as the old codemonitor/code
and codemwait/code instructions. In addition
 codemwaitx/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

 h2a name=5.2GCC 5.2/a/h2

 h2Target Specific Changes/h2

 h3IA-32/x86-64/h3
   ul
 liSupport for new AMD instructions codemonitorx/code and
   codemwaitx/code has been added. This includes new intrinsic
   and built-in support. It is enabled through option 
 code-mmwaitx/code.
   The instructions codemonitorx/code and codemwaitx/code
   implement the same functionality as the old codemonitor/code
   and codemwait/code instructions. In addition codemwaitx/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.




Re: [Patch wwwdocs] gcc-5/changes.html : Document AMD monitorx and mwaitx

2015-07-09 Thread Richard Biener
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
   liSupport for new AMD instructions codemonitorx/code and
   codemwaitx/code has been added. This includes new intrinsic
   and built-in support. It is enabled through option 
 code-mmwaitx/code.
   The instructions codemonitorx/code and codemwaitx/code
   implement the same functionality as the old codemonitor/code
   and codemwait/code instructions. In addition codemwaitx/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.

Richard.

 Regards,
 Venkat.