[PATCH] D41803: ldd::COFF: initalize ErrorHandler with CanExitEarly value

Ben Hamilton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 8 07:58:04 PST 2018


benhamilton added a comment.

> How did you upload this patch, out of curiosity? There are mechanisms in place to automatically add the relevant -commits list as a subscriber now, so I'm wondering why those weren't triggered. CC @benhamilton

Thanks for the reminder! I made https://reviews.llvm.org/H270 for this a while ago, but forgot to add the main LLVM repo. Fixed.

For background, I had to wait until https://reviews.llvm.org/D40502, https://reviews.llvm.org/D40501, https://reviews.llvm.org/D40500, https://reviews.llvm.org/D40499, https://reviews.llvm.org/D40498, https://reviews.llvm.org/D40496, https://reviews.llvm.org/D40495, and https://reviews.llvm.org/D40494 landed before doing so, because otherwise revisions for those repos would all Cc: llvm-commits even if they were for Clang tools, and I forgot to follow up.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D41803





More information about the llvm-commits mailing list