<div dir="ltr">Yea at first I was worried that maybe I changed the semantics of how it looked in PATH, and you had clang-func-mapping in your PATH somewhere before but now lit was building a different PATH.  But I looked at that change and it wasn't even creating that substitution before.  So it looks like that CL is indeed the problem.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 22, 2017 at 11:38 AM Vedant Kumar <<a href="mailto:vsk@apple.com">vsk@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On Sep 22, 2017, at 11:36 AM, Vedant Kumar <<a href="mailto:vsk@apple.com" target="_blank">vsk@apple.com</a>> wrote:<br>
><br>
> Ah, the problem goes away once I build clang-func-mapping.<br>
><br>
> I stripped some stuff out, but here's pretty much what clang/test/<a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a> says my PATH is:<br>
><br>
> ** PATH **:  /Volumes/Builds/llvm.org-coverage-braces-RA/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin<br>
><br>
> I wonder how this ever worked before, when I didn't have clang-func-mapping built.<br>
<br>
Oh, this might be an issue with r313975, which I picked up when rebasing things this morning.<br>
<br>
vedant<br>
<br>
><br>
> Anyway, thanks for your help!<br>
><br>
> vedant<br>
><br>
>> On Sep 22, 2017, at 11:31 AM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
>><br>
>> This works for me.  Can you run "which clang-func-mapping" and also add a line to clang/test/<a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a> to print the value of config.environment['PATH']?<br>
>><br>
>> On Fri, Sep 22, 2017 at 11:27 AM Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
>> Looking, thanks for the report.<br>
>><br>
>> On Fri, Sep 22, 2017 at 11:22 AM Vedant Kumar <<a href="mailto:vsk@apple.com" target="_blank">vsk@apple.com</a>> wrote:<br>
>> As of r313998, this workflow no longer works:<br>
>><br>
>>   cd <build-dir><br>
>>   ./bin/llvm-lit <src>/llvm/tools/clang/test/CoverageMapping<br>
>><br>
>> I get:<br>
>><br>
>> llvm-lit: /Users/vk/src/llvm.org-coverage-braces/llvm/tools/clang/test/<a href="http://lit.cfg.py:97" rel="noreferrer" target="_blank">lit.cfg.py:97</a>: note: using clang: '/Volumes/Builds/llvm.org-coverage-braces-RA/bin/clang'<br>
>> llvm-lit: /Users/vk/src/llvm.org-coverage-braces/llvm/utils/lit/lit/TestingConfig.py:101: fatal: unable to parse config file '/Users/vk/src/llvm.org-coverage-braces/llvm/tools/clang/test/<a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a>', traceback: Traceback (most recent call last):<br>
>>   File "/Users/vk/src/llvm.org-coverage-braces/llvm/utils/lit/lit/TestingConfig.py", line 88, in load_from_path<br>
>>     exec(compile(data, path, 'exec'), cfg_globals, None)<br>
>>   File "/Users/vk/src/llvm.org-coverage-braces/llvm/tools/clang/test/<a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a>", line 128, in <module><br>
>>     config.substitutions.append( ('%clang_func_map', ' ' + lit.util.which('clang-func-mapping', config.environment['PATH']) + ' ') )<br>
>> TypeError: cannot concatenate 'str' and 'NoneType' objects<br>
>><br>
>> However, this still works when running tests within llvm:<br>
>><br>
>>   ./bin/llvm-lit <src>/llvm/test/tools/llvm-cov<br>
>><br>
>> I'd appreciate any help in getting this working again.<br>
>><br>
>> thanks,<br>
>> vedant<br>
>><br>
><br>
<br>
</blockquote></div>