>       [build] More --enable-threads cleanup
>         http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00059.html
> 
> This might well be obvious.

Also, we usually leave those up to the target maintainers, since
they're target specific.  But if it's just a correlation between the
script and a list of source file options, go for it.

> Besides, it would be helpful if a build maintainer could have a look at
> 
>       [build] Move Solaris 2 startup files to toplevel libgcc
>         http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00098.html
> 
> either to state that I don't need approval or to make suggestings for
> improvements.  This is not yet the final patch, but the remainder is
> tuning.

We normally let target maintainers manage anything in the build that's
target-specific.  However, you've mixed in target patches with
target-independent patches.  I have no problem with you checking in
your own target changes, but if you could split out the rest for
review, it would make it easier on us.

Also, mention if you tested it on any ix86 non-solaris platform
(linux, bsd, etc).

Reply via email to