[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
Fri Mar 16 08:18:04 PDT 2018


dstenb added a comment.

A small caveat with this patch is that it does not fix the case where the input file as also added as an extra dependency with -fdepfile-entry; however, I reasoned that it shouldn't really be a problem in practice. I thought that it was a good trade-off ignoring that for slightly simpler code.


Repository:
  rC Clang

https://reviews.llvm.org/D44568





More information about the cfe-commits mailing list