[PATCH] D89761: Split out llvm/Support/FileSystem/UniqueID.h and clang/Basic/FileEntry.h, NFC

[Duncan P. N. Exon Smith via Phabricator via cfe-commits]

dexonsmith added inline comments.


================
Comment at: clang/include/clang/Basic/FileEntry.h:33
+
+using llvm::Optional;
+using llvm::StringRef;
----------------
JDevlieghere wrote:
> Won't this now make `llvm::Optional` visible as `clang::Optional` everywhere this header is included? Isn't this considered bad practice in a header? 
I thought I found both of these usable in clang headers elsewhere. I’ll double check. 


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

https://reviews.llvm.org/D89761