[all-commits] [llvm/llvm-project] 24d0ef: [OpenMP] Fixed a bug when displaying affinity

Shilei Tian via All-commits all-commits at lists.llvm.org
Thu Nov 12 19:27:49 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 24d0ef0f503f8230f115df049ee0ccd067f0881b
      https://github.com/llvm/llvm-project/commit/24d0ef0f503f8230f115df049ee0ccd067f0881b
  Author: Shilei Tian <tianshilei1992 at gmail.com>
  Date:   2020-11-12 (Thu, 12 Nov 2020)

  Changed paths:
    M openmp/runtime/src/kmp_safe_c_api.h
    M openmp/runtime/test/affinity/format/simple_env.c

  Log Message:
  -----------
  [OpenMP] Fixed a bug when displaying affinity

Currently the affinity format string has initial value. When users set
the format via OMP_AFFINITY_FORMAT, it will overwrite the format string. However,
when copying the format, the tailing null is missing. As a result, if the user
format string is shorter than default value, the remaining part in the default
value still makes effort. This bug is not exposed because the test case doesn't
check the end of a string. It only checks whether given output "contains" the
check string.

Reviewed By: AndreyChurbanov

Differential Revision: https://reviews.llvm.org/D91309




More information about the All-commits mailing list