[committed] wwwdocs: gcc-11/changes.html: Editorial changes for RISC-V

Kito Cheng kito.cheng@sifive.com
Mon Jun 14 17:23:35 GMT 2021


Hi Gerald:

Thanks your patch, my grammar is really...weak :p

Hi Bernhard:

>> +  <li>Add new option <code>-misa-spec=*</code> to control ISA spec version.
>> +      This controls the default version of each extensions.
>> +      It defaults to 2.2.</li>
> Is "each extensions" in plural really correct gramatically?
> "default version of extensions" or
> "default version of each extension"
> And is "version of" correct WRT the code or does the code control the
> version "for extensions"?
>
> I.e.
> "to control the default ISA spec version for extensions" if that's what
> the option does?

ISA spec version control a set of default version, like that:

2.2 = { i-ext=2.0, m-ext=2.0, a-ext=2.0 f-ext=2.0 ..}
20190608 = { i-ext=2.1, m-ext=2.0, a-ext=2.0 f-ext=2.2 ...}
20191213 = { i-ext=2.1, m-ext=2.0, a-ext=2.1 f-ext=2.2 ...}

so...I think
"to control the default ISA spec version for extensions"
should be right, feel free to commit patches to fix the
 gramm issue, I will be very grateful :)


On Mon, Jun 14, 2021 at 3:41 PM Gerald Pfeifer <gerald@pfeifer.com> wrote:
>
> On Sun, 13 Jun 2021, Bernhard Reutner-Fischer wrote:
> >> +  <li>Add new option <code>-misa-spec=*</code> to control ISA spec version.
> >> +      This controls the default version of each extensions.
> >> +      It defaults to 2.2.</li>
> > Is "each extensions" in plural really correct gramatically?
> > "default version of extensions" or
> > "default version of each extension"
>
> No, that one escaped my edits - good catch, Bernhard!
>
> > And is "version of" correct WRT the code or does the code control the
> > version "for extensions"?
> >
> > I.e.
> > "to control the default ISA spec version for extensions" if that's what
> > the option does?
>
> Let's see what Kito advises. Once we know, feel free to make a change or
> advise and I'll do.
>
> Thanks,
> Gerald


More information about the Gcc-patches mailing list