r221222 - [llvm-api-change] Use findProgramByName.
Rafael EspĂndola
rafael.espindola at gmail.com
Thu Nov 6 05:14:54 PST 2014
ping. We should change the code or the coding style, but not leave
them like this.
On 3 November 2014 23:55, Rafael EspĂndola <rafael.espindola at gmail.com> wrote:
>> - std::string P(llvm::sys::FindProgramByName(TargetSpecificExecutable));
>> - if (!P.empty())
>> - return P;
>> + auto P = llvm::sys::findProgramByName(TargetSpecificExecutable);
>> + if (P)
>> + return *P;
>> }
>
> It is probably better to write out the type.
>
> Cheers,
> Rafael
More information about the cfe-commits
mailing list