This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: Doc update: Describe new MSP430 feature
- From: Gerald Pfeifer <gerald at pfeifer dot com>
- To: Nick Clifton <nickc at redhat dot com>
- Cc: "Joseph S. Myers" <joseph at codesourcery dot com>, Sandra Loosemore <sandra at codesourcery dot com>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 6 May 2015 00:10:11 +0200 (CEST)
- Subject: Re: RFA: Doc update: Describe new MSP430 feature
- Authentication-results: sourceware.org; auth=none
- References: <87zj5jc9y1 dot fsf at redhat dot com>
Hi Nick,
On Tue, 5 May 2015, Nick Clifton wrote:
> I recently contributes a patch to enhance the MSP430 backend with the
> ability to automatically distribute code and data between high and low
> memory regions. Below is a patch to update the html documentation
> with a description of this new feature. Is it OK to apply ?
as maintainer of the port you do not need anyone else's approval.
That said, I am always happy to provide a second pair of eyes, so
here are some comments:
> Index: htdocs/gcc-5/changes.html
> ===================================================================
> + <li><p>The MSP430 compiler now has the ability to automatically distribute code
> + and data between low (less than 64K) memory and high memory.
Should that be "addresses below ..." or something like that, since
the system would still have more than 64K of memory, not less?
> + This only
> + applies parts
Is a "to" missing here?
> + <p>Two new command line options - <code>-mcode-region=[lower|upper|either]</code>
"command-line options"
Does this really apply to GCC 5, or GCC 6 and the (new) gcc-6/changes.html?
Gerald