[PATCH] D20338: [PCH] Fixed overridden files always invalidating preamble even when unchanged

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 9 13:48:53 PDT 2016


rsmith added inline comments.

================
Comment at: lib/Basic/FileManager.cpp:389
@@ -383,2 +388,3 @@
   UFE->File.reset();
+  UFE->IsVirtual = true;
   return UFE;
----------------
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[]`.


http://reviews.llvm.org/D20338





More information about the cfe-commits mailing list