[PATCH] D26288: Deduplicate replacements by FileEntry instead of file names.

[Manuel Klimek via cfe-commits]

klimek added a comment.

In https://reviews.llvm.org/D26288#586932, @ioeric wrote:

> - Addressed comments: handle non-existing files.


We're not really handling them now though? We're just printing an error?

My point is that we might run the replacement generation on a distributed system, and then group/deduplicate/apply them somewhere where the files might not actually exist (think the reduce stage of a mapreduce). If possible, I'd like to not rely on the existence of the file when we deal with Replacements. I'd find it especially problematic if the existence or non-existence of files changes the semantics of those operations.


https://reviews.llvm.org/D26288