Hi Sandra, > -----Original Message----- > From: Sandra Loosemore [mailto:san...@codesourcery.com] > Sent: Wednesday, November 15, 2017 16:38 > To: Tamar Christina <tamar.christ...@arm.com>; gcc-patches@gcc.gnu.org > Cc: nd <n...@arm.com>; James Greenhalgh <james.greenha...@arm.com>; > Richard Earnshaw <richard.earns...@arm.com>; Marcus Shawcroft > <marcus.shawcr...@arm.com> > Subject: Re: [PATCH][GCC][DOCS][AArch64][ARM] Documentation updates > adding -A extensions. > > On 11/15/2017 04:51 AM, Tamar Christina wrote: > > Hi All, > > > > This patch updates the documentation for AArch64 and ARM correcting > > the use of the architecture namings by adding the -A suffix in appropriate > places. > > Just to clarify, was the documentation previously using incorrect terminology, > or are there new non-A ARMv7 and ARMv8 architectures that invalidate > existing uses of those terms without the -A suffix?
Yes, there are the -M and -R suffixes/profiles. A lot of the documentation was written before these existed. It is mainly a find and replace, but I tried to determine for each change whether the instructions exist in the other profiles. Hopefully they'll all correct but I'll leave that for the review. > And, are the "appropriate > places" all currently-unsuffixed uses, or just a subset of incorrect uses? > It turned out I had to change all of them, for AArch64 for instance we only have A profile. Which is why all unsuffixes changed to -A. For Aarch32 the explicitly different stuff Already had the correct suffixes, so I changed the rest to -A as well. Tamar. > The actual patch looks like search-and-replace to me and I have no objection > to it, but I'd like to understand the rationale so that I can try to remember > what the conventions are for future patch review.... > > -Sandra