On Fri, 19 Feb 2016, Martin Sebor wrote:
> > ... Here I'd like to get my updated patch reviewed so that I
> > can move on to my other GCC 6 tasks.
>
> I integrated the documentation update into the coding patch for bug
> 69780 - [4.9/5/6 Regression] ICE on __builtin_alloca_with_align, to
> keep the two in sync.
>
> Jakub has reviewed and approved the code changes but defers
> the documentation review to you. The most recent combined patch
> is at the link below:
>
> https://gcc.gnu.org/ml/gcc-patches/2016-02/msg01227.html
"doesn't cause it doesn't exceed" doesn't make sense.
I think when you talk about VLAs you should link to the relevant section
of the manual.
I don't think "allocates an object @var{align} bytes large" is correct
(it's size bytes large, not align bytes large, surely?).
"@code{g()}" is contrary to the GNU Coding Standards: "Please do not write
@samp{()} after a function name just to indicate it is a function.
@code{foo ()} is not a function, it is a function call with no
arguments.".
--
Joseph S. Myers
jos...@codesourcery.com
