[llvm] r247518 - [llvm-mc] More meaningful error if input file doesn't exist.
Davide Italiano via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 13 11:45:41 PDT 2015
On Sun, Sep 13, 2015 at 11:08 AM, Davide Italiano <davide at freebsd.org> wrote:
> On Sun, Sep 13, 2015 at 4:43 AM, Rafael EspĂndola
> <rafael.espindola at gmail.com> wrote:
>> testcase?
>>
>
> Will add one.
>
>> Can't this be done by testing the value of EC down at
>> BufferPtr.getError() instead of calling sys::fs::exists?
>>
>> Cheers,
>> Rafael
>>
>
> Probably it could. I was just mimicing what was already there (e.g. llvm-size).
> I will do a sweep over all tools needing this and send out a review,
> it's arguably better.
>
hmm, this turned out to be slightly more annoying than I expected.
It seems we have a very non-uniform error handling among tools for the
same error, so for now I'll just fixup this one and think about others
at a later time.
--
Davide
More information about the llvm-commits
mailing list