Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test case

From	"Wangnan (F)" <wangnan0@huawei.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test case
Date	2016-12-06 03:20 +0100
Message-ID	<sLdwl-4gv-3@gated-at.bofh.it> (permalink)
References	(1 earlier) <sHFrb-3qJ-9@gated-at.bofh.it> <sJYg1-5Hd-13@gated-at.bofh.it> <sL4Mp-71m-25@gated-at.bofh.it> <sL8Gl-1fF-17@gated-at.bofh.it> <sL9j3-1tE-15@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw


On 2016/12/6 5:48, Arnaldo Carvalho de Melo wrote:
> Em Mon, Dec 05, 2016 at 07:02:48PM -0200, Arnaldo Carvalho de Melo escreveu:
>> Em Mon, Dec 05, 2016 at 08:51:01AM -0800, Alexei Starovoitov escreveu:
>>> yeah. it's kinda high. I'm guessing rpm llvm libs are in debug mode.
>>> Try llvm-config --build-mode --assertion-mode
>>> it should be Release OFF
>> Probably this was with 3.9 and built from git, quite a while ago, now I
>> removed it from /usr/local/ and installed what is in f25, but I fear it
>> will be insufficient, does 3.8 cuts it for what we're testing? Humm, it
>> looks like it will:
>>
>> [root@jouet ~]# llc --version
>> LLVM (http://llvm.org/):
>>    LLVM version 3.8.0
>>
>> But I'm now running the container based tests to send a pull req, will
>> check later, after that.
> Not really, Wang, we need to update that feature detection test to state what
> is the minimum required LLVM/clang version, one that has those functions,
> which, unfortunately, isn't the one in the latest fedora, fedora 25:

I'll set the minimum required LLVM version to 3.9, and report
warning when LLVM is too old. However, since LLVM interface is
keep changing, finally we will have problem if we want to support
2 or 3 different clang/LLVM. We should keep moving minimum
requirement LLVM version if we don't want to see '#ifdef's
spread in our code.

Thank you.

Thread

[PATCH v3 10/30] perf clang: Add builtin clang support ant test case Wang Nan <wangnan0@huawei.com> - 2016-11-26 08:20 +0100
  Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case Alexei Starovoitov <alexei.starovoitov@gmail.com> - 2016-11-26 18:20 +0100
    Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case "Wangnan (F)" <wangnan0@huawei.com> - 2016-11-28 08:50 +0100
  Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case Arnaldo Carvalho de Melo <acme@redhat.com> - 2016-12-02 16:50 +0100
    Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case "Wangnan (F)" <wangnan0@huawei.com> - 2016-12-05 03:40 +0100
      Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case "Wangnan (F)" <wangnan0@huawei.com> - 2016-12-05 05:50 +0100
    Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case Alexei Starovoitov <alexei.starovoitov@gmail.com> - 2016-12-05 18:00 +0100
      Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case Arnaldo Carvalho de Melo <acme@redhat.com> - 2016-12-05 22:10 +0100
        Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case Arnaldo Carvalho de Melo <acme@redhat.com> - 2016-12-05 22:50 +0100
          Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case "Wangnan (F)" <wangnan0@huawei.com> - 2016-12-06 03:20 +0100
            Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-12-06 14:50 +0100
      Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-12-06 20:30 +0100
        Re: [PATCH v3 10/30] perf clang: Add builtin clang support ant test  case Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-12-06 22:10 +0100
  [tip:perf/core] perf clang: Add builtin clang support ant test case tip-bot for Wang Nan <tipbot@zytor.com> - 2016-12-06 09:30 +0100

csiph-web