Re: [patch]: PR/39518 missing documentation of cygwin/mingw target options

[Kai Tietz <ktietz70 at googlemail dot com>]

2009/3/23 Dave Korn <dave.korn.cygwin@googlemail.com>:
> Dave Korn wrote:
>> Kai Tietz wrote:
>>> 2009/3/23 Christopher Faylor <cgf-use-the-mailinglist-please@gnu.org>:
>>
>>>> You need to run your changes through a spell checker.
>>
>>> ?
>>>
>>
>>
>> ? Hang on a minute, I'll give it a once-over for spelling and style/grammar/usage.
>
>
>
> ?CAUTION UNTESTED. ?I don't have an up-to-date trunk build around at the
> moment, so I haven't run "make info" on this yet, it needs verification before
> any of it gets installed.
>
> ?As well as tweaking minor language issues, I extended the docs just a little
> to try and summarize the effects defined by the driver specs for these
> options. ?Kai, that might be more than you wanted, feel free to take as much
> or as little of these suggestions as you like :)
>
> ?Oh, also one coding style issue: Kai, there are always supposed to be two
> spaces between the period at the end of a sentence and the start of the next.
>
> ? ?cheers,
> ? ? ?DaveK
>
>
> Index: gcc/gcc/doc/invoke.texi
> ===================================================================
> --- gcc/gcc/doc/invoke.texi ? ? (revision 144856)
> +++ gcc/gcc/doc/invoke.texi ? ? (working copy)
> @@ -826,6 +826,10 @@
> ?@emph{x86-64 Options}
> ?See i386 and x86-64 Options.
>
> +@emph{i386 and x86-64 Windows Options}
> +@gccoptlist{-mconsole -mcygwin -mno-cygwin -mdll
> +-mnop-fun-dllimport -mthread -mwin32 -mwindows}
> +
> ?@emph{Xstormy16 Options}
> ?@gccoptlist{-msim}
>
> @@ -8827,6 +8831,7 @@
> ?* H8/300 Options::
> ?* HPPA Options::
> ?* i386 and x86-64 Options::
> +* i386 and x86-64 Windows Options::
> ?* IA-64 Options::
> ?* M32C Options::
> ?* M32R/D Options::
> @@ -9140,10 +9145,6 @@
> ?the compiler generates code to handle function calls via function
> ?pointers.
>
> -@item -mnop-fun-dllimport
> -@opindex mnop-fun-dllimport
> -Disable support for the @code{dllimport} attribute.
> -
> ?@item -msingle-pic-base
> ?@opindex msingle-pic-base
> ?Treat the register used for PIC addressing as read-only, rather than
> @@ -15460,6 +15461,71 @@
>
> ?These are listed under @xref{i386 and x86-64 Options}.
>
> +@node i386 and x86-64 Windows Options
> +@subsection i386 and x86-64 Windows Options
> +@cindex i386 and x86-64 Windows Options
> +
> +These additional options are available for Windows targets:
> +
> +@table @gcctabopt
> +@item -mconsole
> +@opindex mconsole
> +This option is available for Cygwin and MinGW targets. ?It
> +specifies that a console application is to be generated, by
> +instructing the linker to set the PE header subsystem type
> +required for console applications.
> +This is the default behaviour for Cygwin and MinGW targets.
> +
> +@item -mcygwin
> +@opindex mcygwin
> +This option is available for Cygwin targets. ?It specifies that
> +the Cygwin internal interface is to be used for predefined
> +preprocessor macros, C runtime libraries and related linker
> +paths and options. ?For Cygwin targets this is the default behaviour.
> +This option is deprecated and will be removed in a future release.
> +
> +@item -mno-cygwin
> +@opindex mno-cygwin
> +This option is available for Cygwin targets. ?It specifies that
> +the MinGW internal interface is to be used instead of Cygwin's, by
> +setting MinGW-related predefined macros and linker paths and default
> +library options.
> +This option is deprecated and will be removed in a future release.
> +
> +@item -mdll
> +@opindex mdll
> +This option is available for Cygwin and MinGW targets. ?It
> +specifies that a DLL - a dynamic link library - is to be
> +generated, enabling the selection of the required runtime
> +startup object and entry point.
> +
> +@item -mnop-fun-dllimport
> +@opindex mnop-fun-dllimport
> +This option is available for Cygwin and MinGW targets. ?It
> +specifies that the dllimport attribute should be ignored.
> +
> +@item -mthread
> +@opindex mthread
> +This option is available for MinGW targets. ?It specifies
> +that MinGW-specific thread support is to be used.
> +
> +@item -mwin32
> +@opindex mwin32
> +This option is available for Cygwin and MinGW targets. ?It
> +specifies that the typical Windows pre-defined macros are to
> +be set in the pre-processor, but does not influence the choice
> +of runtime library/startup code.
> +
> +@item -mwindows
> +@opindex mwindows
> +This option is available for Cygwin and MinGW targets. ?It
> +specifies that a GUI application is to be generated by
> +instructing the linker to set the PE header subsystem type
> +appropriately.
> +@end table
> +
> +See also under @xref{i386 and x86-64 Options} for standard options.
> +
> ?@node Xstormy16 Options
> ?@subsection Xstormy16 Options
> ?@cindex Xstormy16 Options
>
>

Hallo Dave,

Thank you very much for reviewing. I tested the build and it works fine.

Ok to apply to trunk?

Cheers,
Kai


-- 
|  (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination

Attachment: cm_doc.diff
Description: Binary data