* RFA: Doc update: Describe new MSP430 feature
@ 2015-05-05 15:37 Nick Clifton
2015-05-05 22:10 ` Gerald Pfeifer
0 siblings, 1 reply; 3+ messages in thread
From: Nick Clifton @ 2015-05-05 15:37 UTC (permalink / raw)
To: gerald, joseph, sandra; +Cc: gcc-patches
Hi Gerald, Hi Joesph, Hi Sandra,
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 ?
Cheers
Nick
Index: htdocs/gcc-5/changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v
retrieving revision 1.120
diff -u -3 -p -r1.120 changes.html
--- htdocs/gcc-5/changes.html 1 May 2015 22:44:45 -0000 1.120
+++ htdocs/gcc-5/changes.html 5 May 2015 15:31:03 -0000
@@ -859,6 +859,27 @@ here</a>.</p>
when compiling for soft-float targets.</li>
</ul>
+<h3 id="msp430">MSP430</h3>
+ <ul>
+ <li><p>The MSP430 compiler now has the ability to automatically distribute code
+ and data between low (less than 64K) memory and high memory. This only
+ applies parts that actually have both memory regions and only if the
+ linker script for the part has been specifically set up to support this
+ feature.</p>
+
+ <p>A new attribute of <code>either</code> can be applied to both functions
+ and data, and this tells the compiler to place the object into low memory
+ if there is room and into high memory otherwise. Two other new attributes
+ - <code>lower</code> and <code>upper</code> - can be used to explicitly
+ state that an object should be placed in the specified memory region. If
+ there is not enough left in that region the compilation will fail.</p>
+
+ <p>Two new command line options - <code>-mcode-region=[lower|upper|either]</code>
+ and <code>-mdata-region=[lower|upper|either]</code> - can be used to tell
+ the compiler what to do with objects that do not have one of these new
+ attributes.</p></li>
+ </ul>
+
<h3 id="nds32">NDS32</h3>
<ul>
<li>The variadic function ABI implementation is now compatible with
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: RFA: Doc update: Describe new MSP430 feature
2015-05-05 15:37 RFA: Doc update: Describe new MSP430 feature Nick Clifton
@ 2015-05-05 22:10 ` Gerald Pfeifer
2015-05-06 9:57 ` Nicholas Clifton
0 siblings, 1 reply; 3+ messages in thread
From: Gerald Pfeifer @ 2015-05-05 22:10 UTC (permalink / raw)
To: Nick Clifton; +Cc: Joseph S. Myers, Sandra Loosemore, gcc-patches
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: RFA: Doc update: Describe new MSP430 feature
2015-05-05 22:10 ` Gerald Pfeifer
@ 2015-05-06 9:57 ` Nicholas Clifton
0 siblings, 0 replies; 3+ messages in thread
From: Nicholas Clifton @ 2015-05-06 9:57 UTC (permalink / raw)
To: Gerald Pfeifer; +Cc: Joseph S. Myers, Sandra Loosemore, gcc-patches
Hi Gerald,
> as maintainer of the port you do not need anyone else's approval.
Thanks - have checked the (editted) patch in.
> That said, I am always happy to provide a second pair of eyes, so
> here are some comments:
Thanks for those too. I have made the appropriate changes following
your suggestions.
> Does this really apply to GCC 5, or GCC 6 and the (new) gcc-6/changes.html?
GCC 6. I had forgotten to run "cvs update -d" in the wwwdocs directory
and so only saw the gcc-5 directory. Doh. I have now applied the patch
to the gcc-6/changes.html file.
Cheers
Nick
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-05-06 9:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-05 15:37 RFA: Doc update: Describe new MSP430 feature Nick Clifton
2015-05-05 22:10 ` Gerald Pfeifer
2015-05-06 9:57 ` Nicholas Clifton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).