Changes between Version 8 and Version 9 of Commentary/Profiling


Ignore:
Timestamp:
Jan 11, 2007 1:05:00 PM (8 years ago)
Author:
kirsten
Comment:

this is just me talking to myself; move it to a separate page

Legend:

Unmodified
Added
Removed
Modified
  • Commentary/Profiling

    v8 v9  
    1111== Ticky-ticky profiling ==  
    1212 
    13 (The following are my notes as I try to get this working again. Once it is working, I'll turn it into something more coherent. -krc) 
    14  
    15 Macros for bumping ticky counters are now defined in [[GhcFile(includes/Cmm.h)]]. Currently, code compiled with the {{{-fticky-ticky}}} flag fails to link because the macros rely on counter variables (things with names like {{{ENT_DYN_IND_ctr}}} being declared, but there are actually no declarations for them. I'll add those declarations to [[GhcFile(includes/RtsExternal.h)]] so I can get something working. Really, there should be something that automatically generates both the macros that are in [[GhcFile(includes/Cmm.h)]] and the declarations for the corresponding variables, so that they stay in sync. 
    16  
    17 Actually, maybe it would make more sense to add a new file, {{{RtsTicky.h}}} or something, which contains only ticky counter declarations (the same declarations that still exist in [[GhcFile(includes/StgTicky.h)]], which isn't used anymore), and that include that from [[GhcFile(includes/RtsExternal.h)]]. 
    18  
    19 No -- put actual declarations for counter variables in another file, {{{TickyCounters.h}}} or something, and include that only from [[GhcFile(rts/Ticky.c)]]; put ''extern'' declarations for those counters in {{{RtsTicky.h}}}, still included from [[GhcFile(includes/RtsExternal.h)]]. Then later we can automatically generate both {{{RtsTicky.h}}} and {{{TickyCounters.h}}}. The reason for this is that the ticky '''macros''' are all over the place and they refer to the ticky counters, so the ticky counters have to be '''declared''' someplace that everyone includes, but of course the actual initializations only need to happen in one place. (Maybe there's a better way to do this...) 
    20  
    21 No, there don't need to be two files; I was confused. Just {{{TickyCounters.h}}}. 
     13([Commentary/Profiling/TickyNotes The following] are my notes as I try to get this working again. Once it is working, I'll turn it into something more coherent. -krc)