]> sourceware.org Git - systemtap.git/commitdiff
fixed prologue comment re. -fomit-frame-pointer
authorJonathan Lebon <jlebon@redhat.com>
Fri, 21 Feb 2014 21:11:13 +0000 (16:11 -0500)
committerJonathan Lebon <jlebon@redhat.com>
Fri, 21 Feb 2014 21:19:10 +0000 (16:19 -0500)
Although the base pointer register is not used when -fomit-frame-pointer
is used, looking at the assembly shows that the address at which we're
done pushing things on the stack does correspond to the line record
following the declaration's line record, so the heuristic indeed works
properly in that case as well. (I previously thought the right spot
should be the same as the entrypc, but hadn't looked closely enough at
the assembly).

dwflpp.cxx

index 64a4334e0dd68eb38d3941f5f183854116089223..d957d7eff77ffd4abfc12bfbe70096ab7717d45b 100644 (file)
@@ -2129,9 +2129,6 @@ dwflpp::resolve_prologue_endings (func_info_map_t & funcs)
       // outputs one). If there is no explicit marker (e.g. GCC does not), we
       // accept a bigger or equal lineno as a prologue end (this catches GCC's
       // 0-line advances).
-      //     Note that this heuristic does not work properly if the program was
-      // compiled with -fomit-stack-pointer (it skips the first line record,
-      // which really is where the function begins).
 
       // We may have to skip a few because some old compilers plop
       // in dummy line records for longer prologues.  If we go too
This page took 0.029597 seconds and 5 git commands to generate.