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