[PATCH] D24970: [ELF] - Fixed crash on invalid input.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 10:15:34 PDT 2016


grimar added a comment.

In https://reviews.llvm.org/D24970#554047, @davide wrote:

> In https://reviews.llvm.org/D24970#554045, @grimar wrote:
>
> > In https://reviews.llvm.org/D24970#554040, @davide wrote:
> >
> > > I think the tests should live in a different directory, yes.
> > >  Also, you might want to try reducing this binary.
> >
> >
> > It is just 480 bytes, does it worth that you think ?
>
>
> If the test can be reduced, I don't see a reason why we shouldn't do it.


We have about 20 binaries already in input folder: 
*.elf, *.o and *.so files, I think it is just hex-edited files.

I`ll try to craft test output using yaml2obj tomorrow for this.


https://reviews.llvm.org/D24970





More information about the llvm-commits mailing list