On Jul 21, 2013, at 2:09 PM, Joseph S. Myers <jos...@codesourcery.com> wrote: > On Sun, 21 Jul 2013, Mike Stump wrote: > >> I've reviewed and applied the gcc/doc changes that were trivial. The >> only patches not applied were the ok->OK patches. > > *For formal documentation* such as gcc/doc, I think changing ok->OK is > appropriate;
I agree. Formal prose is different, so here are the instances that I found in doc/*. Thanks for the review. Index: doc/tm.texi =================================================================== --- doc/tm.texi (revision 201137) +++ doc/tm.texi (working copy) @@ -4959,7 +4959,7 @@ the function prologue. Normally, the pr @cindex tail calls @deftypefn {Target Hook} bool TARGET_FUNCTION_OK_FOR_SIBCALL (tree @var{decl}, tree @var{exp}) -True if it is ok to do sibling call optimization for the specified +True if it is OK to do sibling call optimization for the specified call expression @var{exp}. @var{decl} will be the called function, or @code{NULL} if this is an indirect call. @@ -9861,7 +9861,7 @@ needed. @deftypefn {Target Hook} bool TARGET_FUNCTION_ATTRIBUTE_INLINABLE_P (const_tree @var{fndecl}) @cindex inlining -This target hook returns @code{true} if it is ok to inline @var{fndecl} +This target hook returns @code{true} if it is OK to inline @var{fndecl} into the current function, despite its having target-specific attributes, @code{false} otherwise. By default, if a function has a target specific attribute attached to it, it will not be inlined. Index: target.def =================================================================== --- target.def (revision 201137) +++ target.def (working copy) @@ -1880,7 +1880,7 @@ needed.", DEFHOOK (function_attribute_inlinable_p, "@cindex inlining\n\ -This target hook returns @code{true} if it is ok to inline @var{fndecl}\n\ +This target hook returns @code{true} if it is OK to inline @var{fndecl}\n\ into the current function, despite its having target-specific\n\ attributes, @code{false} otherwise. By default, if a function has a\n\ target specific attribute attached to it, it will not be inlined.", @@ -2529,7 +2529,7 @@ The default value of this hook is based this is an indirect call. */ DEFHOOK (function_ok_for_sibcall, - "True if it is ok to do sibling call optimization for the specified\n\ + "True if it is OK to do sibling call optimization for the specified\n\ call expression @var{exp}. @var{decl} will be the called function,\n\ or @code{NULL} if this is an indirect call.\n\ \n\