lkml.org 
[lkml]   [2012]   [Jan]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRE: [PATCH v2 4/6] ath6kl: get rid of AR_DBG_LVL_CHECK()
Date
From

> We don't need it as debug calls already have a log level and compiler
> should be smart enough to optimise away the code when ath6kl
> debug code is not enabled.

Have you checked?
I think you are assuming that because the called
function collapses, the call will be removed allowing
the loop to be removed.
I suspect this is only true if the function is inlined.
IIRC the normal linkage rules mean that only static
functions are candidates for inlining.

David




\
 
 \ /
  Last update: 2012-01-05 15:17    [from the cache]
©2003-2011 Jasper Spaans