aboutsummaryrefslogtreecommitdiffstats
path: root/tools/perf/builtin-record.c
diff options
context:
space:
mode:
authorArnaldo Carvalho de Melo <acme@redhat.com>2017-02-14 15:04:48 -0300
committerArnaldo Carvalho de Melo <acme@redhat.com>2017-02-14 15:19:18 -0300
commit423d856a4d6ab26a50309fd051f2bdf0e5d00fd6 (patch)
tree72e6fd2c354308fa759c85b4358ab3acaf68555b /tools/perf/builtin-record.c
parentperf bench numa: Make sure dprintf() is not defined (diff)
downloadlinux-dev-423d856a4d6ab26a50309fd051f2bdf0e5d00fd6.tar.xz
linux-dev-423d856a4d6ab26a50309fd051f2bdf0e5d00fd6.zip
perf tests: Synthesize struct instead of using field after variable sized type
End result is the same, its an ABI, so the struct won't change, avoid using a GNU extension, so that we can catch other cases that may be bugs. Caught when building with clang: tests/parse-no-sample-id-all.c:53:20: error: field 'attr' with variable sized type 'struct attr_event' not at the end of a struct or class is a GNU extension [-Werror,-Wgnu-variable-sized-type-not-at-end] struct attr_event attr; ^ 1 error generated. Testing it: # perf test sample_id 24: Parse with no sample_id_all bit set : Ok # 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-e2vs1x771fc208uvxnwcf08b@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-record.c')
0 files changed, 0 insertions, 0 deletions