diff options
| author | Jonathan Peyton <jonathan.l.peyton@intel.com> | 2018-11-28 20:18:06 +0000 |
|---|---|---|
| committer | Jonathan Peyton <jonathan.l.peyton@intel.com> | 2018-11-28 20:18:06 +0000 |
| commit | 1ce776ea2ffe3653d637143f2d1f1a9a2e8f3614 (patch) | |
| tree | d1c2bd7f0076eff07f226354e2959535dc33a99f /openmp/runtime/src/kmp_settings.cpp | |
| parent | f4c0720ad0bcdb895218cfdc35e72eeea15e294d (diff) | |
| download | bcm5719-llvm-1ce776ea2ffe3653d637143f2d1f1a9a2e8f3614.tar.gz bcm5719-llvm-1ce776ea2ffe3653d637143f2d1f1a9a2e8f3614.zip | |
[OpenMP] Minor cleanup of debug code
* Fix calculation of string length.
* Remove NULL-check of pointer which has been dereferenced.
Patch by Andrey Churbanov
Differential Revision: https://reviews.llvm.org/D54948
llvm-svn: 347801
Diffstat (limited to 'openmp/runtime/src/kmp_settings.cpp')
| -rw-r--r-- | openmp/runtime/src/kmp_settings.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/openmp/runtime/src/kmp_settings.cpp b/openmp/runtime/src/kmp_settings.cpp index d855de8e0a5..7cb5d3655fa 100644 --- a/openmp/runtime/src/kmp_settings.cpp +++ b/openmp/runtime/src/kmp_settings.cpp @@ -410,7 +410,7 @@ static void __kmp_stg_parse_par_range(char const *name, char const *value, int *out_range, char *out_routine, char *out_file, int *out_lb, int *out_ub) { - size_t len = KMP_STRLEN(value + 1); + size_t len = KMP_STRLEN(value) + 1; par_range_to_print = (char *)KMP_INTERNAL_MALLOC(len + 1); KMP_STRNCPY_S(par_range_to_print, len + 1, value, len + 1); __kmp_par_range = +1; @@ -418,7 +418,7 @@ static void __kmp_stg_parse_par_range(char const *name, char const *value, __kmp_par_range_ub = INT_MAX; for (;;) { unsigned int len; - if ((value == NULL) || (*value == '\0')) { + if (*value == '\0') { break; } if (!__kmp_strcasecmp_with_sentinel("routine", value, '=')) { |
