This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH] [asan] Fix for PR asan/56330
On Sat, Feb 16, 2013 at 09:44:51AM +0100, Dodji Seketeli wrote:
> Right. My first bootstrap actually caught this, I updated the patch
> locally to just modify that test and sent out the wrong one. Sorry for
> this. This is the patch that actually passed bootstrap.
Ok. Actually, please change gcc/testsuite/c-c++-common/asan/pr56330.c
to the same thing, I've just verified it ICEs the same way without the patch
even when it is
int d = __builtin_memcmp (s.a, e, 100);
d += __builtin_memcmp (s.a, e, 200);
__builtin_memcpy (s.a, e, 100);
__builtin_memcpy (s.a, e, 200);
it doesn't ICE without the patch).
While pr56330.c test look to be redundant, it is at least tortured at all
compilation options, so it tests something that the other tests don't.
No need to rebootstrap/retest.