This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH/Merge Request] Vtable Verification feature.
- From: "Joseph S. Myers" <joseph at codesourcery dot com>
- To: Ramana Radhakrishnan <ramrad01 at arm dot com>
- Cc: <cmtice at google dot com>, <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 8 Aug 2013 15:28:57 +0000
- Subject: Re: [PATCH/Merge Request] Vtable Verification feature.
- References: <CABtf2+Q47X0Ld6byo8R96TTKuYPkOsKO7tADRRWVpFqHHTsfuw at mail dot gmail dot com> <20130801131950 dot 527bbf97 at oakwood> <CAD_=9DThid3P8ridA2OxP15L_KehaKzw=EGjSPwP=MT6vMAexg at mail dot gmail dot com> <CABtf2+TDrQObufSt4Zd46fjG6jH_XfuhRTeNJvsniP6BaCyEdg at mail dot gmail dot com> <CABtf2+Rn8Xn12xvWMx9FSBZ3EGaZQ77Qs-sh_z1XCxntsN9SVg at mail dot gmail dot com> <CAD_=9DTNJuR-SrjKYAEHLoCYr2OZe8kboei9Xyc2Jug+x1=v9w at mail dot gmail dot com> <20130806143904 dot 3819d0f1 at oakwood> <52039548 dot 4010801 at arm dot com>
On Thu, 8 Aug 2013, Ramana Radhakrishnan wrote:
> I spent some time this morning looking into what it would take to enable this
> for arm*-*-linux* today and the first thing that I noticed is that the
> testsuite is essentially driven by a shell script that caters to native
> testing on the only port that this was developed for. It also expects -m32 and
> -m64 to be a standard option in all ports that want to turn this on. Also
> because the tests are not using dejagnu it's going to be a right pain to get
> this to work for cross-testing and unless that works reliably one isn't going
> to be able to get this feature working easily.
Clearly it's got to be converted to DejaGnu (and must avoid depending on a
build directory at all, installed testing with runtest --tool libvtv,
having created an appropriate site.exp, should work as well). It's
looking rather like this whole feature hasn't been adequately reviewed
before commit....
--
Joseph S. Myers
joseph@codesourcery.com