Ping?  Could somebody please review this for me?

-- Caroline Tice
cmt...@google.com

On Wed, Aug 28, 2013 at 2:15 PM, Caroline Tice <cmt...@google.com> wrote:
> Discussion on this topic seems to have petered out.  I have removed
> references to ENABLE_VTABLE_VERIFY from libvtv, as requested.  I
> cannot remove them from libgcc, as they really need to be there.  I
> have updated the documentation in install.texi, as requested.
> Please review the attached patch, which does all of this, and let me
> know if it is ok to commit.
>
> -- Caroline Tice
> cmt...@google.com
>
> gcc ChangeLog:
>
> 2013-08-28  Caroline Tice  <cmt...@google.com>
>
>         * doc/install.texi: Add documentation for the --enable-vtable-verify
>         and the --disable-libvtv configure options.
>
> libvtv ChangeLog:
> 2013-08-28  Caroline Tice  <cmt...@google.com>
>
>         * Makefile.am: Remove #if ENABLE_VTABLE_VERIFY checks around
>         definitions of SUBDIRS, libvtv_la_SOURCES and libvtv_include_HEADERS.
>         * Makefile.in: Regenerate.
>         * configure.ac: Remove checks and tests for --enable-vtable-verify.
>         * configure: Regenerate.
>
> On Thu, Aug 8, 2013 at 3:11 PM, Joseph S. Myers <jos...@codesourcery.com> 
> wrote:
>> On Thu, 8 Aug 2013, Caroline Tice wrote:
>>
>>> The attached patch adds documentation for the --enable-vtable-verify
>>> configure option to  install.texi.  Is this ok to commit?
>>
>> Could you please answer the questions I raised in the first four
>> paragraphs of <http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00427.html> at
>> greater length?  This patch appears to document it just as a libstdc++
>> configure option, but as noted there it appears to affect more than
>> libstdc++ (even though it would be more sensible if it *only* affected
>> libstdc++, and even better if it just enabled extra multilibs).
>>
>> --
>> Joseph S. Myers
>> jos...@codesourcery.com

Reply via email to