[wwwdocs] Mention treatment of MIPS DSP Control Register

[wwwdocs] Mention treatment of MIPS DSP Control Register

[Catherine Moore]

This mentions the new treatment of the MIPS DSP Control Register by the compiler.
The corresponding patch was posted here:
http://gcc.gnu.org/ml/gcc-patches/2008-10/msg01306.html
Does this look okay to install?  Thanks,
Catherine


cvs diff: Diffing .
Index: changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.4/changes.html,v
retrieving revision 1.38
diff -r1.38 changes.html
363a364,365
 >     <li>The SCOUNT and POS bits of the DSP control register are now
 >         treated as global.</li>

Re: [wwwdocs] Mention treatment of MIPS DSP Control Register

[Richard Sandiford]

Catherine Moore <clm@codesourcery.com> writes:
> This mentions the new treatment of the MIPS DSP Control Register by the compiler.
> The corresponding patch was posted here:
> http://gcc.gnu.org/ml/gcc-patches/2008-10/msg01306.html
> Does this look okay to install?  Thanks,
> Catherine
>
>
> cvs diff: Diffing .
> Index: changes.html
> ===================================================================
> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.4/changes.html,v
> retrieving revision 1.38
> diff -r1.38 changes.html
> 363a364,365
>  >     <li>The SCOUNT and POS bits of the DSP control register are now
>  >         treated as global.</li>

The text looks good, but I think it would be clearer if we compare
this to previous releases.  How about adding something like:

   Previous versions of GCC treated these fields as call-clobbered
   instead.

?

I gather from the line numbers that it's part of the MIPS section.
I think it should go in "Caveats" instead, in which case we should
say "MIPS DSP control register".

Richard

Re: [wwwdocs] Mention treatment of MIPS DSP Control Register

[Catherine Moore]

Hi Richard,
Here's the updated patch which incorporates your suggestions.
I tried to check this in, but I don't seem to have write access to the wwwdocs repository.
Catherine

cvs diff: Diffing .
Index: changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.4/changes.html,v
retrieving revision 1.38
diff -r1.38 changes.html
36a37,40
 >     <li>The SCOUNT and POS bits of the MIPS DSP control register are now
 >     treated as global. Previous versions of GCC treated these fields as
 >     call-clobbered instead.</li>
 >


Richard Sandiford wrote:
> Catherine Moore <clm@codesourcery.com> writes:
>> This mentions the new treatment of the MIPS DSP Control Register by the compiler.
>> The corresponding patch was posted here:
>> http://gcc.gnu.org/ml/gcc-patches/2008-10/msg01306.html
>> Does this look okay to install?  Thanks,
>> Catherine
>>
>>
>> cvs diff: Diffing .
>> Index: changes.html
>> ===================================================================
>> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.4/changes.html,v
>> retrieving revision 1.38
>> diff -r1.38 changes.html
>> 363a364,365
>>  >     <li>The SCOUNT and POS bits of the DSP control register are now
>>  >         treated as global.</li>
> 
> The text looks good, but I think it would be clearer if we compare
> this to previous releases.  How about adding something like:
> 
>    Previous versions of GCC treated these fields as call-clobbered
>    instead.
> 
> ?
> 
> I gather from the line numbers that it's part of the MIPS section.
> I think it should go in "Caveats" instead, in which case we should
> say "MIPS DSP control register".
> 
> Richard

Re: [wwwdocs] Mention treatment of MIPS DSP Control Register

[Gerald Pfeifer]

Hi Catherine,

On Mon, 3 Nov 2008, Catherine Moore wrote:
> I tried to check this in, but I don't seem to have write access to the 
> wwwdocs repository.

if you have access to SVN, CVS should work as well.  Have you had the
chance to look at the instructions at http://gcc.gnu.org/cvs.html on
how to check out a tree for read/write access?

If you continue to have problems, let me know off-list and I'll be happy 
to work this out with you (or just commit the patch on your behalf).

Gerald