[llvm] r257395 - Add a missing error handling to llvm-lto.

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 13:57:10 PST 2016


Fixed in r257475. Sorry about that.

Cheers,
Rafael


On 12 January 2016 at 02:57, Ismail Donmez <ismail at i10z.com> wrote:
> Hi,
>
> On Tue, Jan 12, 2016 at 12:08 AM, Rafael Espindola via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> Author: rafael
>> Date: Mon Jan 11 16:08:22 2016
>> New Revision: 257395
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=257395&view=rev
>> Log:
>> Add a missing error handling to llvm-lto.
>>
>> Added:
>>     llvm/trunk/test/tools/llvm-lto/error.ll
>> Modified:
>>     llvm/trunk/tools/llvm-lto/llvm-lto.cpp
>>
>> Added: llvm/trunk/test/tools/llvm-lto/error.ll
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-lto/error.ll?rev=257395&view=auto
>> ==============================================================================
>> --- llvm/trunk/test/tools/llvm-lto/error.ll (added)
>> +++ llvm/trunk/test/tools/llvm-lto/error.ll Mon Jan 11 16:08:22 2016
>> @@ -0,0 +1,2 @@
>> +; RUN: not llvm-lto foobar 2>&1 | FileCheck %s
>> +; CHECK: llvm-lto: error loading file 'foobar': No such file or directory
>
> This fails on Windows because the error message is "no such file or
> directory" with a small n. Not sure how you want to handle that case.
>
> Regards,
> ismail


More information about the llvm-commits mailing list