lkml.org 
[lkml]   [2015]   [Jun]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC PATCH v7 23/37] perf tools: Introduce llvm config options
From
Date


发自我的 iPhone

> 在 2015年6月13日,上午10:31,Alexei Starovoitov <ast@plumgrid.com> 写道:
>
>> On 6/11/15 10:35 PM, Wang Nan wrote:
>> # Path to clang. If omit, search it from $PATH.
>> clang-path = "/path/to/clang"
>
> I think this bit and search_program() from the next patch is
> overly flexible. It's always delicate to search file paths.

About this clang-path: in my environment the name of clang is x86_64-oe-linux-clang, and installed in /usr/bin/llvm-3.7, neither in $PATH nor has its default name. This is the default setting of yocto and open-embedded based systems. Please see do_install part in building script of it:

https://github.com/openembedded/meta-oe/blob/master/meta-oe/recipes-core/llvm/llvm.inc

Therefore, I think something like this option should not be dropped, or people like me have to do some nasty work to make it useable. Could you please rethink it?

Thank you.

> Unless this is really needed, I would drop this bit.
> popen() in the next patch executes clang via shell,
> so explicit PATH search in unnecessary.
>
> The rest of the patches look good to me.
> Thanks again.
>



\
 
 \ /
  Last update: 2015-06-13 07:21    [W:0.187 / U:0.464 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site