[Bug releng/31683] add --enable-debug configury option

[mcermak at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=31683

--- Comment #5 from Martin Cermak <mcermak at redhat dot com> ---
(In reply to Martin Cermak from comment #4)
> Mulling over the valgrind part..

Seems like currently (as of commit 97e0c4514) valgrind covers translator pass 2
with memcheck (via enumname.exp) and translator pass 4 with helgrind (via
parallelism-helgrind.exp).

The most valuable opportunities for extending the valgrind coverage seem to be:
- memcheck on passes 3 and 4
- memcheck on staprun

BUT zooming out a little bit ... Covering this bug with a dedicated testcase
most likely isn't needed, because if a problem is found, the instrumented code
aborts.  So that's covered.

One thing I've noticed is that --enable-debug requires --without-dyninst
otherwise build fails: Stock dyninst libs are not instrumented, and attempt to
link them fails (missing debug symbols).

That said, what might make sense to do as part of this bug would be to auto
imply --without-dyninst if --enable-debug is set.

Another option is to mark this bug as RESOLVED/FIXED :)  Thoughts?

-- 
You are receiving this mail because:
You are the assignee for the bug.