[PATCH] D69090: [Try 2] Include sanitize blacklist and other extra deps as part of scan-deps output

Mon Oct 21 12:59:06 PDT 2019

kousikk added a comment.

I was able to test this on a Windows machine. The `ExtraDeps` are already being added to the dependency output (this code does that - https://github.com/llvm/llvm-project/blob/master/clang/lib/Frontend/DependencyFile.cpp#L188). My suspicion for why I wasn't seeing them previously was that these ExtraDeps are added as relative paths and my consumer code (which integrates with scan-deps) was ignoring relative paths. I can handle that in the consumer code - I'll leave the behaviour of the tool as it is.

Sorry about the back and forth on this. I'm closing this out.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69090