[PATCH] D35987: [ELF] - Introduce std::vector<InputFile *> global arrays.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 31 05:49:50 PDT 2017


grimar added a comment.

In https://reviews.llvm.org/D35987#824273, @ruiu wrote:

> Hmm, this also looks odd to me. At least it's not straightforward. You put everything in one bag and then search from the bag, while you could separate them from beginning


What do you think about updated version ? (btw it can be reduced slightly if we land https://reviews.llvm.org/D35936 first, though this can be done afterward if we land it too).


https://reviews.llvm.org/D35987





More information about the llvm-commits mailing list