Re: [PATCH v9 1/3] perf report: Support LLVM for addr2line()

[Arnaldo Carvalho de Melo <acme@kernel.org>]

On Sun, Sep 01, 2024 at 12:57:25PM +0200, Steinar H. Gunderson wrote:
> On Tue, Jul 30, 2024 at 04:49:12PM -0300, Arnaldo Carvalho de Melo wrote:
> > Unfortunately it clashed with recent patches in the capstone related
> > codebase, IIRC Athira's for powerpc data-type profiling.
> > 
> > […]
> > 
> > I'll continue processing other patched and eventually try to fix this if
> > you're busy,
> 
> Hi,
> 
> Is there any movement in getting v10 merged? :-) I haven't heard back
> since I sent out the rebase

Test building is detecting some problems, I'll try to address them:

perfbuilder@number:~$ time dm
   1    13.50 almalinux:8                   : FAIL gcc version 8.5.0 20210514 (Red Hat 8.5.0-22) (GCC) 
    util/srcline.c: In function 'dso__free_a2l':
    util/srcline.c:184:20: error: parameter name omitted
     void dso__free_a2l(struct dso *)
                        ^~~~~~~~~~~~
    make[3]: *** [/git/perf-6.11.0-rc3/tools/build/Makefile.build:158: util] Error 2
   2    68.46 almalinux:9                   : FAIL clang version 17.0.6 (AlmaLinux OS Foundation 17.0.6-5.el9)
    util/srcline.c:184:32: error: omitting the parameter name in a function definition is a C2x extension [-Werror,-Wc2x-extensions]
      184 | void dso__free_a2l(struct dso *)
          |                                ^
    1 error generated.
    make[3]: *** [/git/perf-6.11.0-rc3/tools/build/Makefile.build:158: util] Error 2
   3: almalinux:9-i386WARNING: image platform (linux/386) does not match the expected platform (linux/amd64)
WARNING: image platform (linux/386) does not match the expected platform (linux/amd64)
    17.58 almalinux:9-i386              : FAIL gcc version 11.4.1 20231218 (Red Hat 11.4.1-3) (GCC) 
    util/llvm-c-helpers.cpp: In function ‘char* make_symbol_relative_string(dso*, const char*, u64, u64)’:
    util/llvm-c-helpers.cpp:150:52: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 5 has type ‘u64’ {aka ‘long long unsigned int’} [-Werror=format=]
      150 |                 snprintf(buf, sizeof(buf), "%s+0x%lx",
          |                                                  ~~^
          |                                                    |
          |                                                    long unsigned int
          |                                                  %llx
      151 |                          demangled ? demangled : sym_name, addr - base_addr);
          |                                                            ~~~~~~~~~~~~~~~~
          |                                                                 |
          |                                                                 u64 {aka long long unsigned int}
    cc1plus: all warnings being treated as errors
    make[3]: *** [/git/perf-6.11.0-rc3/tools/build/Makefile.build:158: util] Error 2
   4    18.08 alpine:3.15                   : FAIL gcc version 10.3.1 20211027 (Alpine 10.3.1_git20211027) 
    util/srcline.c: In function 'dso__free_a2l':
    util/srcline.c:184:20: error: parameter name omitted
      184 | void dso__free_a2l(struct dso *)
          |                    ^~~~~~~~~~~~
    make[3]: *** [/git/perf-6.11.0-rc3/tools/build/Makefile.build:158: util] Error 2