|
From: Will S. <wil...@vn...> - 2015-05-11 19:14:55
|
On Mon, 2015-05-11 at 20:28 +0200, Matthias Schwarzott wrote: > On 11.05.2015 19:14, Will Schmidt wrote: > > add a leak_cpp_interior test .exp entry to match the current ppc64* output. > > In contrast to the existing .exp entries, there are multiple changes > > throughout the generated output, this one is not filterable. > > > It looks like the multipleinheritance heuristic does not work on this > platform. > Maybe the code in aligned_ptr_above_page0_is_vtable_addr in > mc_leakcheck.c is wrong. > But at least it has: > #if defined(VGA_ppc64be) || defined(VGA_ppc64le) Ok. That actually seems to be the problem with this test. ppc64LE does not use the function descriptors like BE does. Thanks for pointing out where exactly to look. :-) locally, dropping the defined(VGA_ppc64le) portion of the ifdef appears to clear the failure up. I'll do a clean respin to verify and check for side affects and re-submit patch 6/7 shortly. Thanks, -Will > Regards > Matthias > |