[PATCH] D43336: [ELF] Add ICF test case to symbol-ordering-file-warnings.s

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 15 07:50:18 PST 2018


jhenderson added a comment.

In https://reviews.llvm.org/D43336#1008776, @grimar wrote:

> I also don't know. See https://reviews.llvm.org/D43234. Warning seems good idea for me. 
>  Does it make sence to disable ICF for sections involved in ordering ?


Thanks, I didn't see that review. I'll post my thoughts on it over there.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D43336





More information about the llvm-commits mailing list