[PATCH] D44568: Fix emission of phony dependency targets when adding extra deps
David Stenberg via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 29 05:56:07 PDT 2018
dstenb added a comment.
In https://reviews.llvm.org/D44568#1114188, @vsapsai wrote:
> Looks good to me. Just watch the build bots in case some of them are strict with warnings and require `(void)AddFilename(Filename)`.
Yes, I'll keep an eye out for that.
I'll submit this shortly. Thanks for the review!
https://reviews.llvm.org/D44568
More information about the cfe-commits
mailing list