[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