Author: sewardj
Date: 2007-10-14 18:25:22 +0100 (Sun, 14 Oct 2007)
New Revision: 6994

Log:
More tuning.

Modified:
   branches/THRCHECK/thrcheck/tc_main.c


Modified: branches/THRCHECK/thrcheck/tc_main.c
===================================================================
--- branches/THRCHECK/thrcheck/tc_main.c        2007-10-14 13:42:27 UTC (rev 
6993)
+++ branches/THRCHECK/thrcheck/tc_main.c        2007-10-14 17:25:22 UTC (rev 
6994)
@@ -3062,7 +3062,7 @@
 }
 
 
-static void cacheline_wback ( UWord way, UWord wix )
+static __attribute__((noinline)) void cacheline_wback ( UWord way, UWord wix )
 {
    Word        i, j;
    Bool        anyShared = False;
@@ -3164,7 +3164,7 @@
    associated with 'wix' is assumed to have already been filled in;
    hence that is used to determine where in the backing store to read
    from. */
-static void cacheline_fetch ( UWord way, UWord wix )
+static __attribute__((noinline)) void cacheline_fetch ( UWord way, UWord wix )
 {
    Word        i;
    Addr        tag;
@@ -3249,9 +3249,9 @@
    return (UWord)(a & (N_LINE_W8s - 1));
 }
 
-static CacheLine* get_cacheline_MISS ( Addr a ); /* fwds */
-__attribute__((noinline))
-static /*inline*/ CacheLine* get_cacheline ( Addr a )
+static __attribute__((noinline))
+       CacheLine* get_cacheline_MISS ( Addr a ); /* fwds */
+static inline CacheLine* get_cacheline ( Addr a )
 {
    /* tag is 'a' with the in-line offset masked out, 
       eg a[31]..a[4] 0000 */
@@ -3266,7 +3266,8 @@
    return get_cacheline_MISS( a );
 }
 
-static CacheLine* get_cacheline_MISS ( Addr a )
+static __attribute__((noinline))
+       CacheLine* get_cacheline_MISS ( Addr a )
 {
    /* tag is 'a' with the in-line offset masked out, 
       eg a[31]..a[4] 0000 */
@@ -3365,7 +3366,7 @@
    if (UNLIKELY(is_SHVAL_Invalid(svOld))) {
       tl_assert(svOld == SHVAL_InvalidU);
       pulldown_to_w8( cl, ix8 );
-      tl_assert(is_sane_CacheLine(cl));
+      /* EXPENSIVE: tl_assert(is_sane_CacheLine(cl)); */
       svOld = cl->w8[ix8];
    }
    svNew = msm__handle_read( thr_acc, a, svOld );
@@ -3383,7 +3384,7 @@
    if (UNLIKELY(is_SHVAL_Invalid(svOld))) {
       if (svOld == SHVAL_InvalidD) goto slowcase;
       pulldown_to_w16( cl, ix16 );
-      tl_assert(is_sane_CacheLine(cl));
+      /* EXPENSIVE: tl_assert(is_sane_CacheLine(cl)); */
       svOld = cl->w16[ix16];
    }
    svNew = msm__handle_read( thr_acc, a, svOld );
@@ -3406,7 +3407,7 @@
    if (UNLIKELY(is_SHVAL_Invalid(svOld))) {
       if (svOld == SHVAL_InvalidD) goto slowcase;
       pulldown_to_w32( cl, ix32 );
-      tl_assert(is_sane_CacheLine(cl));
+      /* EXPENSIVE: tl_assert(is_sane_CacheLine(cl)); */
       svOld = cl->w32[ix32];
    }
    svNew = msm__handle_read( thr_acc, a, svOld );
@@ -3450,7 +3451,7 @@
    if (UNLIKELY(is_SHVAL_Invalid(svOld))) {
       tl_assert(svOld == SHVAL_InvalidU);
       pulldown_to_w8( cl, ix8 );
-      tl_assert(is_sane_CacheLine(cl));
+      /* EXPENSIVE: tl_assert(is_sane_CacheLine(cl)); */
       svOld = cl->w8[ix8];
    }
    svNew = msm__handle_write( thr_acc, a, svOld );
@@ -3468,7 +3469,7 @@
    if (UNLIKELY(is_SHVAL_Invalid(svOld))) {
       if (svOld == SHVAL_InvalidD) goto slowcase;
       pulldown_to_w16( cl, ix16 );
-      tl_assert(is_sane_CacheLine(cl));
+      /* EXPENSIVE: tl_assert(is_sane_CacheLine(cl)); */
       svOld = cl->w16[ix16];
    }
    svNew = msm__handle_write( thr_acc, a, svOld );
@@ -3491,7 +3492,7 @@
    if (UNLIKELY(is_SHVAL_Invalid(svOld))) {
       if (svOld == SHVAL_InvalidD) goto slowcase;
       pulldown_to_w32( cl, ix32 );
-      tl_assert(is_sane_CacheLine(cl));
+      /* EXPENSIVE: tl_assert(is_sane_CacheLine(cl)); */
       svOld = cl->w32[ix32];
    }
    svNew = msm__handle_write( thr_acc, a, svOld );
@@ -3535,7 +3536,7 @@
    if (UNLIKELY(is_SHVAL_Invalid(svOld))) {
       tl_assert(svOld == SHVAL_InvalidU);
       pulldown_to_w8( cl, ix8 );
-      tl_assert(is_sane_CacheLine(cl));
+      /* EXPENSIVE: tl_assert(is_sane_CacheLine(cl)); */
       svOld = cl->w8[ix8];
       tl_assert(is_SHVAL_valid(svOld));
    }
@@ -3635,7 +3636,7 @@
    if (UNLIKELY(is_SHVAL_Invalid(sv))) {
       tl_assert(sv == SHVAL_InvalidU);
       pulldown_to_w8( cl, ix8 );
-      tl_assert(is_sane_CacheLine(cl));
+      /* tl_assert(is_sane_CacheLine(cl)); */
       sv = cl->w8[ix8];
    }
    tl_assert(is_SHVAL_valid(sv));


-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Valgrind-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/valgrind-developers

Reply via email to