Re: [Valgrind-developers] valgrind: r8990 - in branches/DARWIN: coregrind coregrind/m_aspacemgr coregrind/m_debuginfo coregrind/m_replacemalloc coregrind/m_scheduler coregrind/m_syswrap massif memcheck

[Bart V. A. <bar...@gm...>]

On Wed, Jan 21, 2009 at 1:25 PM, Julian Seward <js...@ac...> wrote:
> On Tuesday 20 January 2009, Nicholas Nethercote wrote:
>> On Wed, Jan 21, 2009 at 12:08 AM, Florian Krohm <br...@ac...> wrote:
>> > On Monday 19 January 2009 23:16:09 sv...@va... wrote:
>> >> -      VG_(printf)("%d %d\n", *thin_size, sizeof(struct MACH_HEADER));
>> >> +      VG_(printf)("%d %lu\n", *thin_size, sizeof(struct MACH_HEADER));
>> >
>> > C99 has %zu for arguments of type size_t. It's possible, though, that
>> > some older versions of gcc -Wformat don't handle this. What is the
>> > minimum GCC version that you assume for compilation ?
>>
>> We still have a --workaround-gcc296-bugs flag...
>
> That really means "ignore accesses in the 256 or so bytes below %esp"
> (just pretend you didn't see them).  Maybe it should be renamed.
>
> Minimum version of gcc now is 3.0.x, I believe.

Please don't try to compile Valgrind with gcc 2.9.x -- at least the
DRD source code is compiled incorrectly with this compiler version,
probably due to an optimizer bug.

Bart.