<div dir="ltr"><div style="font-family:courier new,monospace;font-size:large" class="gmail_default">Now I have to learn to use Phabricator :)</div><div class="gmail_default" style="font-family:courier new,monospace;font-size:large"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 12, 2020 at 8:26 AM Artem Dergachev <<a href="mailto:noqnoqneo@gmail.com">noqnoqneo@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Yes, perl scan-build is absolutely still used, so patches are very welcome!<br>
<br>
Regex, or just extract the filename, or maybe even *always* set these <br>
variables because it's unlikely to make things worse.<br>
<br>
On 2/12/20 1:58 AM, Csaba Ráduly via cfe-dev wrote:<br>
> Hi all,<br>
><br>
> scan-build looks at the name of the build command to decide whether to <br>
> add CC=... and CXX=... commandline options for make-like build <br>
> commands, in order to override these values if they are set in the <br>
> makefile (which wins over environment variables).<br>
><br>
> <a href="https://github.com/llvm/llvm-project/blob/master/clang/tools/scan-build/bin/scan-build#L1118" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/blob/master/clang/tools/scan-build/bin/scan-build#L1118</a> <br>
><br>
><br>
> Unfortunately, this means that<br>
><br>
> scan-build /usr/local/bin/make or scan-build $HOME/bin/gmake<br>
><br>
> are not recognized as "make-like" and CC=... and CXX=... options are <br>
> not added.<br>
><br>
> This check should be made with a regex, just like the checks before.<br>
><br>
> Is this Perl script still used? Should I make a patch?<br>
><br>
> Csaba<br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">You can get very substantial performance improvements<br>by not doing the right thing. - Scott Meyers, An Effective C++11/14 Sampler<br>So if you're looking for a completely portable, 100% standards-conformant way<br>to get the wrong information: this is what you want. - Scott Meyers (C++TDaWYK)</div>