|
From: <sv...@va...> - 2013-01-16 22:07:13
|
philippe 2013-01-16 22:07:02 +0000 (Wed, 16 Jan 2013)
New Revision: 13235
Log:
Small comment fix: .h specifies "all frames", implementation uses 8.
Two fixes could be done:
Either we fix the comments
or we increase N_FRAMES to be rather VG_DEEPEST_BACKTRACE.
We fix the comment for the following reason:
This is (at least for the moment) not performance critical.
as this is only called when an error is reported.
However, searching for local vars is extremely costly.
It is unlikely that an error is reported for a stack variable
which is more than 8 frames deeper than theframe in which
it is detected.
So, fix the comment, waiting for a complaint that a deeper
variable is not properly described.
Modified files:
trunk/coregrind/m_debuginfo/debuginfo.c
trunk/include/pub_tool_debuginfo.h
Modified: trunk/include/pub_tool_debuginfo.h (+1 -1)
===================================================================
--- trunk/include/pub_tool_debuginfo.h 2013-01-16 03:18:19 +00:00 (rev 13234)
+++ trunk/include/pub_tool_debuginfo.h 2013-01-16 22:07:02 +00:00 (rev 13235)
@@ -96,7 +96,7 @@
/*OUT*/PtrdiffT* offset );
/* Try to form some description of DATA_ADDR by looking at the DWARF3
- debug info we have. This considers all global variables, and all
+ debug info we have. This considers all global variables, and 8
frames in the stacks of all threads. Result is written at the ends
of DNAME{1,2}V, which are XArray*s of HChar, that have been
initialised by the caller, and True is returned. If no description
Modified: trunk/coregrind/m_debuginfo/debuginfo.c (+1 -1)
===================================================================
--- trunk/coregrind/m_debuginfo/debuginfo.c 2013-01-16 03:18:19 +00:00 (rev 13234)
+++ trunk/coregrind/m_debuginfo/debuginfo.c 2013-01-16 22:07:02 +00:00 (rev 13235)
@@ -3171,7 +3171,7 @@
}
/* Try to form some description of DATA_ADDR by looking at the DWARF3
- debug info we have. This considers all global variables, and all
+ debug info we have. This considers all global variables, and 8
frames in the stacks of all threads. Result is written at the ends
of DNAME{1,2}V, which are XArray*s of HChar, that have been
initialised by the caller, and True is returned. If no description
|