[COMMITTED] arc: Add ARC entry for gcc-10/changes.html

Message ID 20200303091214.1437978-1-claziss@gmail.com
State New
Headers show
Series
  • [COMMITTED] arc: Add ARC entry for gcc-10/changes.html
Related show

Commit Message

Claudiu Zissulescu March 3, 2020, 9:12 a.m.
Add ARC entry for gcc-10/changes.html

---
 htdocs/gcc-10/changes.html | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

-- 
2.24.1

Comments

Martin Sebor March 3, 2020, 4:47 p.m. | #1
On 3/3/20 2:12 AM, Claudiu Zissulescu wrote:
> Add ARC entry for gcc-10/changes.html

> 

> ---

>   htdocs/gcc-10/changes.html | 14 +++++++++++++-

>   1 file changed, 13 insertions(+), 1 deletion(-)

> 

> diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html

> index 53d0ca08..4e27c05b 100644

> --- a/htdocs/gcc-10/changes.html

> +++ b/htdocs/gcc-10/changes.html

> @@ -557,7 +557,19 @@ a work-in-progress.</p>

>     much improved.</li>

>   </ul>

>   

> -<!-- <h3 id="arc">ARC</h3> -->

> +<h3 id="arc">ARC</h3>

> +<ul>

> +  <li>The interrupt service routine functions saves all used

                                                  ^^^^^
Just a small typo: they "save".

> +  registers, including extension registers and auxiliary registers

> +  used by Zero Overhead Loops.</li>

> +  <li>Improve code-size by using multiple short instructions instead


Not terribly important but I don't think the hyphen in code size is
called for (it's not used as an adjective).

Martin

> +  of a single long mov or ior instruction when its long immediate

> +  constant is known.</li>

> +  <li>Fix usage of the accumulator register for ARC600.</li>

> +  <li>Fix issues with <code>uncached</code> attribute.</li>

> +  <li>Remove <code>-mq-class</code> option.</li>

> +  <li>Improve 64-bit integer addition and subtraction operations.</li>

> +</ul>

>   

>   <h3 id="arm">arm</h3>

>   <ul>

>
Claudiu Zissulescu March 4, 2020, 8:47 a.m. | #2
Done 🙂 Thank you for your review,
Claudiu
________________________________
From: Martin Sebor <msebor@gmail.com>

Sent: Tuesday, March 3, 2020 6:47 PM
To: Claudiu Zissulescu <claziss@gmail.com>; gcc-patches@gcc.gnu.org <gcc-patches@gcc.gnu.org>
Cc: gerald@pfeifer.com <gerald@pfeifer.com>; law@redhat.com <law@redhat.com>; Francois Bedard <fbedard@synopsys.com>; Claudiu Zissulescu <claziss@synopsys.com>; andrew.burgess@embecosm.com <andrew.burgess@embecosm.com>
Subject: Re: [PATCH] [COMMITTED] arc: Add ARC entry for gcc-10/changes.html

On 3/3/20 2:12 AM, Claudiu Zissulescu wrote:
> Add ARC entry for gcc-10/changes.html

>

> ---

>   htdocs/gcc-10/changes.html | 14 +++++++++++++-

>   1 file changed, 13 insertions(+), 1 deletion(-)

>

> diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html

> index 53d0ca08..4e27c05b 100644

> --- a/htdocs/gcc-10/changes.html

> +++ b/htdocs/gcc-10/changes.html

> @@ -557,7 +557,19 @@ a work-in-progress.</p>

>     much improved.</li>

>   </ul>

>

> -<!-- <h3 id="arc">ARC</h3> -->

> +<h3 id="arc">ARC</h3>

> +<ul>

> +  <li>The interrupt service routine functions saves all used

                                                  ^^^^^
Just a small typo: they "save".

> +  registers, including extension registers and auxiliary registers

> +  used by Zero Overhead Loops.</li>

> +  <li>Improve code-size by using multiple short instructions instead


Not terribly important but I don't think the hyphen in code size is
called for (it's not used as an adjective).

Martin

> +  of a single long mov or ior instruction when its long immediate

> +  constant is known.</li>

> +  <li>Fix usage of the accumulator register for ARC600.</li>

> +  <li>Fix issues with <code>uncached</code> attribute.</li>

> +  <li>Remove <code>-mq-class</code> option.</li>

> +  <li>Improve 64-bit integer addition and subtraction operations.</li>

> +</ul>

>

>   <h3 id="arm">arm</h3>

>   <ul>

>
Gerald Pfeifer March 7, 2020, 9:05 p.m. | #3
On Tue, 3 Mar 2020, Claudiu Zissulescu wrote:
> Add ARC entry for gcc-10/changes.html


What do you think of the following refinement to mark up 
mov and ior as code?

Gerald

PS: And thanks for your other feedback, Martin!

diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html
index 710e3863..5ba388cf 100644
--- a/htdocs/gcc-10/changes.html
+++ b/htdocs/gcc-10/changes.html
@@ -562,9 +562,9 @@ a work-in-progress.</p>
   <li>The interrupt service routine functions save all used
   registers, including extension registers and auxiliary registers
   used by Zero Overhead Loops.</li>
-  <li>Improve code size by using multiple short instructions instead
-  of a single long mov or ior instruction when its long immediate
-  constant is known.</li>
+  <li>Improve code size by using multiple short instructions instead of
+  a single long <code>mov</code> or <code>ior</code> instruction when its
+  long immediate constant is known.</li>
   <li>Fix usage of the accumulator register for ARC600.</li>
   <li>Fix issues with <code>uncached</code> attribute.</li>
   <li>Remove <code>-mq-class</code> option.</li>
Claudiu Zissulescu March 9, 2020, 10:31 a.m. | #4
It looks great 🙂 I'll keep in mind this tip next time.

Thank you,
Clauidu
________________________________
From: Gerald Pfeifer <gerald@pfeifer.com>

Sent: Saturday, March 7, 2020 11:05 PM
To: Claudiu Zissulescu <claziss@gmail.com>; Martin Sebor <msebor@gmail.com>
Cc: gcc-patches@gcc.gnu.org <gcc-patches@gcc.gnu.org>; Jeff Law <law@redhat.com>; Francois Bedard <fbedard@synopsys.com>; Claudiu Zissulescu <claziss@synopsys.com>; andrew.burgess@embecosm.com <andrew.burgess@embecosm.com>
Subject: Re: [PATCH] [COMMITTED] arc: Add ARC entry for gcc-10/changes.html

On Tue, 3 Mar 2020, Claudiu Zissulescu wrote:
> Add ARC entry for gcc-10/changes.html


What do you think of the following refinement to mark up
mov and ior as code?

Gerald

PS: And thanks for your other feedback, Martin!

diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html
index 710e3863..5ba388cf 100644
--- a/htdocs/gcc-10/changes.html
+++ b/htdocs/gcc-10/changes.html
@@ -562,9 +562,9 @@ a work-in-progress.</p>
   <li>The interrupt service routine functions save all used
   registers, including extension registers and auxiliary registers
   used by Zero Overhead Loops.</li>
-  <li>Improve code size by using multiple short instructions instead
-  of a single long mov or ior instruction when its long immediate
-  constant is known.</li>
+  <li>Improve code size by using multiple short instructions instead of
+  a single long <code>mov</code> or <code>ior</code> instruction when its
+  long immediate constant is known.</li>
   <li>Fix usage of the accumulator register for ARC600.</li>
   <li>Fix issues with <code>uncached</code> attribute.</li>
   <li>Remove <code>-mq-class</code> option.</li>

Patch

diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html
index 53d0ca08..4e27c05b 100644
--- a/htdocs/gcc-10/changes.html
+++ b/htdocs/gcc-10/changes.html
@@ -557,7 +557,19 @@  a work-in-progress.</p>
   much improved.</li>
 </ul>
 
-<!-- <h3 id="arc">ARC</h3> -->
+<h3 id="arc">ARC</h3>
+<ul>
+  <li>The interrupt service routine functions saves all used
+  registers, including extension registers and auxiliary registers
+  used by Zero Overhead Loops.</li>
+  <li>Improve code-size by using multiple short instructions instead
+  of a single long mov or ior instruction when its long immediate
+  constant is known.</li>
+  <li>Fix usage of the accumulator register for ARC600.</li>
+  <li>Fix issues with <code>uncached</code> attribute.</li>
+  <li>Remove <code>-mq-class</code> option.</li>
+  <li>Improve 64-bit integer addition and subtraction operations.</li>
+</ul>
 
 <h3 id="arm">arm</h3>
 <ul>