[PATCH] D20338: [PCH] Fixed overridden files always invalidating preamble even when unchanged
Cameron via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 9 15:00:34 PDT 2016
cameron314 added inline comments.
================
Comment at: lib/Basic/FileManager.cpp:389
@@ -383,2 +388,3 @@
UFE->File.reset();
+ UFE->IsVirtual = true;
return UFE;
----------------
rsmith wrote:
> Yes. The `IsValid` flag is just supposed to mean that this file has actually been added to the `UniqueRealFiles` map rather than having been default-constructed by `operator[]`.
Excellent then, I'll get rid of `IsVirtual` and use `IsValid` in place. This will condense things down to a one-line change plus a large diff for the test ^^
http://reviews.llvm.org/D20338
More information about the cfe-commits
mailing list