This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [wwwdocs] add news items for TM work


Hej Aldy,

thanks for doing this.  It is mostly okay, I just have a minor
suggestion or two.

On Tue, 15 Nov 2011, Aldy Hernandez wrote:
> + The project was partially funded by
> + the <a href="http://www.velox-project.eu/";>Velox</a> project.  This
> + feature is experimental and is available for C and C++ on selected
> + platforms.

Reading this I was wondering which platforms.  On the other page it
says x86 and alpha; should we just use this here, or were you thinking
to keep this more marketing speak? ;-)

> Index: gcc-4.7/changes.html
> ===================================================================
> +   <li>Experimental support for transactional memory has been added.
> +       It includes support for the compiler, as well as a supporting
> +       runtime library called <code>libitm</code>.  To compile code
> +       with transactional memory constructs, use
> +       the <code>-fgnu-tm</code> option.

Put this under <p>...</p> as well, I think.

> + 	Support is currently available for the x86 and Alpha platforms.

32-bit, 64-bit x86, or both?

> +       <p>
> + 	This work was contributed by Red Hat and was partly funded by
> + 	the <a href="http://www.velox-project.eu/";>Velox</a> project.
> +       </p>

In the release notes we generally don't have those acknowledgements,
only in the News section.

Gerald


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]