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

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 10:05:41 PDT 2016


davide added a comment.

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.


https://reviews.llvm.org/D24970





More information about the llvm-commits mailing list