[PATCH] D22079: Refactor Archive-child iteration.

Lang Hames via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 8 12:46:06 PDT 2016


lhames added a comment.

> That would leave clients unable to detect the difference between a broken archive and an undefined symbol. I don't think we want to do that.


More accurately - dropping the Error argument would leave us unable to tell the difference, but if I understand correctly dropping the error was the main motivation for switching to Optional.

- Lang.


Repository:
  rL LLVM

http://reviews.llvm.org/D22079





More information about the llvm-commits mailing list