Re: RFR: 8152538: UL doesn't inline the LogTagSet::is_level check

2016-03-23 Thread Bengt Rutisson
Hi StefanK, Looks good! Bengt On 2016-03-23 11:28, Stefan Karlsson wrote: Hi all, Please, review this patch to optimize the log_is_enabled check in the Unified Logging framework. http://cr.openjdk.java.net/~stefank/8152538/webrev.01/ https://bugs.openjdk.java.net/browse/JDK-8152538 The f

Re: RFR: 8152538: UL doesn't inline the LogTagSet::is_level check

2016-03-23 Thread Marcus Larsson
Hi Stefan, On 03/23/2016 11:28 AM, Stefan Karlsson wrote: Hi all, Please, review this patch to optimize the log_is_enabled check in the Unified Logging framework. http://cr.openjdk.java.net/~stefank/8152538/webrev.01/ Looks good, thanks for fixing this! Marcus https://bugs.openjdk.java.

RFR: 8152538: UL doesn't inline the LogTagSet::is_level check

2016-03-23 Thread Stefan Karlsson
Hi all, Please, review this patch to optimize the log_is_enabled check in the Unified Logging framework. http://cr.openjdk.java.net/~stefank/8152538/webrev.01/ https://bugs.openjdk.java.net/browse/JDK-8152538 The following code: void logLine() { log_debug(gc)("logLine"); } expands in