On 11/18/14 09:48, Ilya Enkovich wrote:
On 15 Nov 00:10, Jeff Law wrote:
On 11/14/14 10:26, Ilya Enkovich wrote:
Hi,

This patch introduces a simple library with several wrappers to be used with 
MPX and Pointer Bounds Checker.  Wrappers allow to obtain, copy and just keep 
alive bounds whrough widely use library calls.  It significantly increases 
checking  quality.

Thanks,
Ilya
--
gcc/

2014-11-14  Ilya Enkovich  <ilya.enkov...@intel.com>

        * gcc.c (MPX_SPEC): Add wrappers library.

libmpx/

2014-11-14  Ilya Enkovich  <ilya.enkov...@intel.com>

        * Makefile.am (SUBDIRS): New.
        (MAKEOVERRIDES): New.
        * Makefile.in: Regenerate.
        * configure.ac: Add mpxintr/Makefile to config
        files.
        * configure: Regenerate.
        * mpxwrap/Makefile.am: New.
        * mpxwrap/Makefile.in: New.
        * mpxwrap/libtool-version: New.
        * mpxwrap/mpx_wrappers.cc: New.
As Joseph mentioned, symbol versioning.  Anytime a target side
library is added to GCC, it should be properly versioned.

Don't forget copyright headers in the new files.  Remember it has to
be suitable for embeddeding in the target without infecting the
target with the GPL.  LGPL or GPL + exception clause seem the most
appropriate to me.


Jeff


Thank you for review!  Here is a version with license and versioning added.

Thanks,
Ilya
--
gcc/

2014-11-18  Ilya Enkovich  <ilya.enkov...@intel.com>

        * gcc.c (MPX_SPEC): Add wrappers library.

libmpx/

2014-11-18  Ilya Enkovich  <ilya.enkov...@intel.com>

        * Makefile.am (SUBDIRS): New.
        (MAKEOVERRIDES): New.
        * Makefile.in: Regenerate.
        * configure.ac: Add mpxintr/Makefile to config
        files.
        * configure: Regenerate.
        * mpxwrap/Makefile.am: New.
        * mpxwrap/Makefile.in: New.
        * mpxwrap/libtool-version: New.
        * mpxwrap/mpx_wrappers.cc: New.
        * mpxwrap/libmpxwrappers.map: New.
OK.
Jeff

Reply via email to