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

Jonas Devlieghere via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 19 22:20:20 PDT 2020


JDevlieghere added inline comments.


================
Comment at: clang/include/clang/Basic/FileEntry.h:33
+
+using llvm::Optional;
+using llvm::StringRef;
----------------
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? 


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

https://reviews.llvm.org/D89761



More information about the cfe-commits mailing list