[PATCH] D25025: [ELF] - Do not crash on invalid symbol index.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 3 03:30:03 PDT 2016


>since both tests hit the same check, why do you need 2?
>
>Cheers,
>Rafael

Initially I did that because they have different roots of problem. 
First one just has incorrect type of .symtab section, second has incorrect
symbol index.
But since both are hitting the same check using common code path it probably was excessive,
I'll remove one of testcases and update the patch.

George.


More information about the llvm-commits mailing list