rsmith added inline comments.

================
Comment at: lib/Basic/FileManager.cpp:389
@@ -383,2 +388,3 @@
   UFE->File.reset();
+  UFE->IsVirtual = true;
   return UFE;
----------------
Rather than adding this `IsVirtual` flag, could you just set `UFE->IsValid` to 
`true` here?  It looks like a simple oversight that this code fails to set the 
`IsValid` flag properly.


Repository:
  rL LLVM

http://reviews.llvm.org/D20338



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to