On Thu, Aug 31, 2023 at 11:14:13AM +0800, Yujie Yang wrote:
> On Wed, Aug 30, 2023 at 09:36:22PM +0000, Joseph Myers wrote:
> > On Wed, 30 Aug 2023, Yang Yujie wrote:
> >
> > > +A suffix @code{[/ARCH][/OPTION]...]} may follow immediately after the ABI
> > > +identifier to customize the compiler options for building the given set
> > > of
> > > +libraries. @code{ARCH} denotes the architecture name recognized by the
> > > +@code{-march=ARCH} compiler option, which acts as a basic target ISA
> > > +configuration that can be adjusted using the subsequent @code{OPTION}
> > > +suffixes, where each @code{OPTION} is a compiler option itself.
> >
> > Since ARCH and OPTION are not literal strings of program source code, you
> > should actually be using @var{arch} and @var{option} for them (and @dots{}
> > instead of ..., since the ... isn't literal source code either).
> >
> > This patch series also adds a new configure option --with-strict-align-lib
> > that needs documenting in the corresponding patch.
> >
> > --
> > Joseph S. Myers
> > jos...@codesourcery.com
>
> Thanks for the review. Does the following fix look good?
> If so, I will include these in the patchset.
Found something else to fix, I will post them in the next version
of the patchset.
