[Bug target/80057] typo in mips.opt: Virtualization Application Specific

2017-04-10 Thread mpf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80057

mpf at gcc dot gnu.org changed:

   What|Removed |Added

 Status|WAITING |RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from mpf at gcc dot gnu.org ---
Fixed on trunk.

[Bug target/80057] typo in mips.opt: Virtualization Application Specific

2017-04-10 Thread mpf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80057

--- Comment #4 from mpf at gcc dot gnu.org ---
Author: mpf
Date: Mon Apr 10 13:44:39 2017
New Revision: 246807

URL: https://gcc.gnu.org/viewcvs?rev=246807&root=gcc&view=rev
Log:
Update MIPS -mvirt option description

gcc/
PR target/80057
* config/mips/mips.opt (-mvirt): Update description.
* doc/invoke.texi (-mvirt): Likewise.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/mips/mips.opt
trunk/gcc/doc/invoke.texi

[Bug target/80057] typo in mips.opt: Virtualization Application Specific

2017-03-31 Thread mpf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80057

mpf at gcc dot gnu.org changed:

   What|Removed |Added

 CC||mpf at gcc dot gnu.org

--- Comment #3 from mpf at gcc dot gnu.org ---
I agree there is an inconsistency here (or several actually but not all can be
fixed).

We refer to the Virtualization Application-Specific Extension with shortened
forms of Virtualization ASE or VZ ASE.

Unfortunately the option name -mvirt is not -mvz but I can't fix that. We can
fix the description though so it matches terms used elsewhere:

Use Virtualization (VZ) instructions.

I'll push a patch for it.

[Bug target/80057] typo in mips.opt: Virtualization Application Specific

2017-03-15 Thread roland.illig at gmx dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80057

--- Comment #2 from Roland Illig  ---
Citing from the PDF document:

> The Virtualization Application-Specific Extension (Module) requires the 
> following base architecture support:

The current GCC code split these words at the following boundaries:

> The "Virtualization Application Specific" "Module"

This would imply that there is a module called VAS.

The usual reading is different, though:

> The "Virtualization" "application-specific module"

This means there is an application-specific module called "Virtualization".

The words "Application-Specific Extension" are probably only capitalized
because they have been defined somewhere as a concept. In GCC, these words
should not be capitalized, and there should be a hyphen between
"application-specific", to be consistent with the grammar in other places.

[Bug target/80057] typo in mips.opt: Virtualization Application Specific

2017-03-15 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80057

Andrew Pinski  changed:

   What|Removed |Added

 Status|UNCONFIRMED |WAITING
   Last reconfirmed||2017-03-15
 Ever confirmed|0   |1

--- Comment #1 from Andrew Pinski  ---
https://imagination-technologies-cloudfront-assets.s3.amazonaws.com/documentation/MD00849-2B-VZmicroMIPS64-AFP-01.06.pdf


"The Virtualization Application-Specific Extension (Module) requires the
following base architecture support:"

Or did I misunderstand your issue with it?