[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