diff mbox

[wwwdocs] Improve arm and aarch64-related info in readings.html

Message ID 573DDB63.40101@foss.arm.com
State New
Headers show

Commit Message

Kyrill Tkachov May 19, 2016, 3:27 p.m. UTC
Hi all,

I noticed that we have a readings.html page that has pointers to documentation of various backends that GCC supports.
The info on arm seems a bit out of date and somewhat confusing, and there is no entry for aarch64.
This patch tries to address that.

The arm entry is updated to not mention armv2(?) and thumb and an aarch64 entry is added with
a link to the ARM documentation.

Ok to commit?

Thanks,
Kyrill

Comments

James Greenhalgh May 23, 2016, 8:55 a.m. UTC | #1
On Thu, May 19, 2016 at 04:27:31PM +0100, Kyrill Tkachov wrote:
> Hi all,
> 
> I noticed that we have a readings.html page that has pointers to
> documentation of various backends that GCC supports.  The info on arm seems a
> bit out of date and somewhat confusing, and there is no entry for aarch64.
> This patch tries to address that.
> 
> The arm entry is updated to not mention armv2(?) and thumb and an aarch64
> entry is added with a link to the ARM documentation.
> 
> Ok to commit?

The AArch64 part looks fine to me, but a paragraph introducing what is
meant by AArch64 ("the 64-bit execution state introduced in the ARMv8-A
architecture profile" - or similar) would be a useful addition. Just
sending our users straight to the full repository of information provided
by ARM may give them too wide a set of readings :).

Thanks,
James

> ? readings.html~
> Index: readings.html
> ===================================================================
> RCS file: /cvs/gcc/wwwdocs/htdocs/readings.html,v
> retrieving revision 1.242
> diff -U 3 -r1.242 readings.html
> --- readings.html	14 Nov 2015 23:40:21 -0000	1.242
> +++ readings.html	15 Feb 2016 13:29:03 -0000
> @@ -59,6 +59,11 @@
>  </p>
>  
>  <ul>
> + <li>AArch64
> +  <br /><a href="http://infocenter.arm.com/help/index.jsp">
> +	ARM Documentation</a>
> + </li>
> +
>   <li>alpha
>     <br />Manufacturer: Compaq (DEC)
>     <br /><a href="http://www.tru64unix.compaq.com/docs/base_doc/DOCUMENTATION/V51A_HTML/ARH9MBTE/TITLE.HTM">Calling
> @@ -81,12 +86,12 @@
>    <br /><a href="http://www.synopsys.com/IP/PROCESSORIP/ARCPROCESSORS/Pages/default.aspx">ARC Documentation</a>
>   </li>
>  
> - <li>arm (armv2, thumb)
> -  <br />Manufacturer: Various, by license from ARM
> + <li>ARM
> +  <br />Manufacturer: Various, by license from ARM.
>    <br />CPUs include: ARM7 and ARM7T series (eg. ARM7TDMI), ARM9 and StrongARM
>    <br /><a href="http://infocenter.arm.com/help/index.jsp">ARM Documentation</a>
>   </li>
> - 
> +
>   <li>AVR
>    <br />Manufacturer: Atmel
>    <br /><a href="http://www.atmel.com/products/microcontrollers/avr/">AVR Documentation</a>
Gerald Pfeifer May 28, 2016, 8:04 p.m. UTC | #2
Hi Kyrill,

On Thu, 19 May 2016, Kyrill Tkachov wrote:
> I noticed that we have a readings.html page that has pointers to 
> documentation of various backends that GCC supports. The info on arm 
> seems a bit out of date and somewhat confusing, and there is no entry 
> for aarch64. This patch tries to address that.

I see you have not applied this yet.  In case you were waiting
for any further approvals beyond James': (a) His is sufficient
for doc and web pages around AArch64, and (b) I agree, too. :-)

Gerald
Kyrill Tkachov May 31, 2016, 8:15 a.m. UTC | #3
Hi Gerald,

On 28/05/16 21:04, Gerald Pfeifer wrote:
> Hi Kyrill,
>
> On Thu, 19 May 2016, Kyrill Tkachov wrote:
>> I noticed that we have a readings.html page that has pointers to
>> documentation of various backends that GCC supports. The info on arm
>> seems a bit out of date and somewhat confusing, and there is no entry
>> for aarch64. This patch tries to address that.
> I see you have not applied this yet.  In case you were waiting
> for any further approvals beyond James': (a) His is sufficient
> for doc and web pages around AArch64, and (b) I agree, too. :-)

I was going to apply it (with James's suggestions) but I got
sidetracked by other things. I'll do it today.

Thanks,
Kyrill

> Gerald
diff mbox

Patch

? readings.html~
Index: readings.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/readings.html,v
retrieving revision 1.242
diff -U 3 -r1.242 readings.html
--- readings.html	14 Nov 2015 23:40:21 -0000	1.242
+++ readings.html	15 Feb 2016 13:29:03 -0000
@@ -59,6 +59,11 @@ 
 </p>
 
 <ul>
+ <li>AArch64
+  <br /><a href="http://infocenter.arm.com/help/index.jsp">
+	ARM Documentation</a>
+ </li>
+
  <li>alpha
    <br />Manufacturer: Compaq (DEC)
    <br /><a href="http://www.tru64unix.compaq.com/docs/base_doc/DOCUMENTATION/V51A_HTML/ARH9MBTE/TITLE.HTM">Calling
@@ -81,12 +86,12 @@ 
   <br /><a href="http://www.synopsys.com/IP/PROCESSORIP/ARCPROCESSORS/Pages/default.aspx">ARC Documentation</a>
  </li>
 
- <li>arm (armv2, thumb)
-  <br />Manufacturer: Various, by license from ARM
+ <li>ARM
+  <br />Manufacturer: Various, by license from ARM.
   <br />CPUs include: ARM7 and ARM7T series (eg. ARM7TDMI), ARM9 and StrongARM
   <br /><a href="http://infocenter.arm.com/help/index.jsp">ARM Documentation</a>
  </li>
- 
+
  <li>AVR
   <br />Manufacturer: Atmel
   <br /><a href="http://www.atmel.com/products/microcontrollers/avr/">AVR Documentation</a>