perf symbols: No need to check if sym->name is NULL 83/292583/1
authorArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 13 Feb 2017 19:52:15 +0000 (16:52 -0300)
committerSeung-Woo Kim <sw0312.kim@samsung.com>
Tue, 9 May 2023 09:23:42 +0000 (18:23 +0900)
commit02ca6eb5cbc1dbadba0659a7eb859ea88ede9db1
treed0885d8297138a322e374fa40a86ea98c55a8950
parent6107bc80dc077ad62dd5355c3c710d69af912c58
perf symbols: No need to check if sym->name is NULL

As it is an array, so will always evaluate to 'true', as reported by
clang:

  builtin-sched.c:2070:19: error: address of array 'sym->name' will always evaluate to 'true' [-Werror,-Wpointer-bool-conversion]
                  if (sym && sym->name) {
                          ~~ ~~~~~^~~~
  1 warning generated.

So just ditch all those useless checks.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-ydpm927col06paixb775jjx5@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
[sw0312.kim: backport upstream commit a7c3899c0686 to resolve gcc-12 build issue]
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Change-Id: I512d780fa0d3420ee4d24d61f1fcdc78c6ed73bc
tools/perf/builtin-kmem.c
tools/perf/util/machine.c
tools/perf/util/symbol_fprintf.c