Message ID | 20141217144901.GA24101@gmail.com |
---|---|
State | New |
Headers | show |
On Wednesday 2014-12-17 06:49, H.J. Lu wrote: > Index: gcc-5/changes.html > =================================================================== > RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v > retrieving revision 1.52 > diff -u -p -r1.52 changes.html > --- gcc-5/changes.html 15 Dec 2014 19:55:08 -0000 1.52 > +++ gcc-5/changes.html 17 Dec 2014 14:48:21 -0000 > @@ -444,6 +444,13 @@ void operator delete[] (void *, std::siz > place of the __fentry__ or mcount call, so that a call per function > can be later patched in. This can be used for low overhead tracing or > hot code patching.</li> > + <li> The new <code>-malign-data=</code> option to control how > + GCC aligns variables. Let's make this "...option controls how..." This is fine with this change and considering the genuine question below. > <code>-malign-data=compat</code> uses > + increased alignment value compatible with GCC 4.8 and earlier, > + <code>-malign-data=abi</code> uses alignment value as specified by > + the psABI, and <code>-malign-data=cacheline</code> uses increased > + alignment value to match the cache line size. > + <code>-malign-data=compat</code> is the default.</li> Here, and in the .texi documentation, would it be appropriate to just say "alignment" instead of "alignment value" throughout, or is there particular reason to say the latter? Gerald
On Thu, Dec 25, 2014 at 8:56 PM, Gerald Pfeifer <gerald@pfeifer.com> wrote: > On Wednesday 2014-12-17 06:49, H.J. Lu wrote: >> Index: gcc-5/changes.html >> =================================================================== >> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v >> retrieving revision 1.52 >> diff -u -p -r1.52 changes.html >> --- gcc-5/changes.html 15 Dec 2014 19:55:08 -0000 1.52 >> +++ gcc-5/changes.html 17 Dec 2014 14:48:21 -0000 >> @@ -444,6 +444,13 @@ void operator delete[] (void *, std::siz >> place of the __fentry__ or mcount call, so that a call per function >> can be later patched in. This can be used for low overhead tracing or >> hot code patching.</li> >> + <li> The new <code>-malign-data=</code> option to control how >> + GCC aligns variables. > > Let's make this "...option controls how..." Done. > This is fine with this change and considering the genuine question > below. > >> <code>-malign-data=compat</code> uses >> + increased alignment value compatible with GCC 4.8 and earlier, >> + <code>-malign-data=abi</code> uses alignment value as specified by >> + the psABI, and <code>-malign-data=cacheline</code> uses increased >> + alignment value to match the cache line size. >> + <code>-malign-data=compat</code> is the default.</li> > > Here, and in the .texi documentation, would it be appropriate to > just say "alignment" instead of "alignment value" throughout, or > is there particular reason to say the latter? I don't have a strong opinion on it. Please feel free to improve it. Thanks.
Index: gcc-5/changes.html =================================================================== RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v retrieving revision 1.52 diff -u -p -r1.52 changes.html --- gcc-5/changes.html 15 Dec 2014 19:55:08 -0000 1.52 +++ gcc-5/changes.html 17 Dec 2014 14:48:21 -0000 @@ -444,6 +444,13 @@ void operator delete[] (void *, std::siz place of the __fentry__ or mcount call, so that a call per function can be later patched in. This can be used for low overhead tracing or hot code patching.</li> + <li> The new <code>-malign-data=</code> option to control how + GCC aligns variables. <code>-malign-data=compat</code> uses + increased alignment value compatible with GCC 4.8 and earlier, + <code>-malign-data=abi</code> uses alignment value as specified by + the psABI, and <code>-malign-data=cacheline</code> uses increased + alignment value to match the cache line size. + <code>-malign-data=compat</code> is the default.</li> </ul> <h3 id="sh">SH</h3>