[PATCH] D145101: [clang][deps] NFC: Simplify worker loop

Ben Langmuir via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 1 13:17:06 PST 2023


benlangmuir accepted this revision.
benlangmuir added a comment.
This revision is now accepted and ready to land.

> I guess it was originally done this way because we have a number of DependencyScanning{Tool,Worker} instances

Oh of course, this makes sense.  Yeah we could maybe find a different way to do it if we care, but this is a sufficiently good reason to keep doing what we're doing and shouldn't hold you up.

Thanks for explaining; LGTM.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D145101/new/

https://reviews.llvm.org/D145101



More information about the cfe-commits mailing list