This is an archive of the discontinued LLVM Phabricator instance.

Eliminate double printing of verbose affinity settings
ClosedPublic

Authored by jlpeyton on Nov 29 2017, 2:56 PM.

Reviewers

AndreyChurbanov
tlwilmar

Commits

Summary

Redundant extra verbose output of binding to full mask in case affinity=balanced or OMP_PLACES=<any> or OMP_PROC_BIND=<any>

LGTM

This revision is now accepted and ready to land.Nov 29 2017, 3:01 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

runtime/

src/

4 lines

Show First 20 Lines • Show All 4,461 Lines • ▼ Show 20 Lines	#else
} else {		} else {
KA_TRACE(100, ("__kmp_affinity_set_init_mask: binding T#%d to mask %d\n",		KA_TRACE(100, ("__kmp_affinity_set_init_mask: binding T#%d to mask %d\n",
gtid, i));		gtid, i));
}		}
#endif /* OMP_40_ENABLED */		#endif /* OMP_40_ENABLED */

KMP_CPU_COPY(th->th.th_affin_mask, mask);		KMP_CPU_COPY(th->th.th_affin_mask, mask);

if (__kmp_affinity_verbose) {		if (__kmp_affinity_verbose
		/* to avoid duplicate printing (will be correctly printed on barrier) */
		&& (__kmp_affinity_type == affinity_none \|\| i != KMP_PLACE_ALL)) {
char buf[KMP_AFFIN_MASK_PRINT_LEN];		char buf[KMP_AFFIN_MASK_PRINT_LEN];
__kmp_affinity_print_mask(buf, KMP_AFFIN_MASK_PRINT_LEN,		__kmp_affinity_print_mask(buf, KMP_AFFIN_MASK_PRINT_LEN,
th->th.th_affin_mask);		th->th.th_affin_mask);
KMP_INFORM(BoundToOSProcSet, "KMP_AFFINITY", (kmp_int32)getpid(),		KMP_INFORM(BoundToOSProcSet, "KMP_AFFINITY", (kmp_int32)getpid(),
__kmp_gettid(), gtid, buf);		__kmp_gettid(), gtid, buf);
}		}

#if KMP_OS_WINDOWS		#if KMP_OS_WINDOWS
▲ Show 20 Lines • Show All 566 Lines • Show Last 20 Lines