[clang] 125f445 - [clang] Remove clang::Optional

[Kazu Hirata via cfe-commits]

Author: Kazu Hirata
Date: 2023-01-29T18:08:00-08:00
New Revision: 125f4457a54a550846732763ee36b1447ec8d66e

URL: https://github.com/llvm/llvm-project/commit/125f4457a54a550846732763ee36b1447ec8d66e
DIFF: https://github.com/llvm/llvm-project/commit/125f4457a54a550846732763ee36b1447ec8d66e.diff

LOG: [clang] Remove clang::Optional

This is part of an effort to migrate from llvm::Optional to
std::optional:

https://discourse.llvm.org/t/deprecating-llvm-optional-x-hasvalue-getvalue-getvalueor/63716

Differential Revision: https://reviews.llvm.org/D142857

Added: 
    

Modified: 
    clang/include/clang/Basic/LLVM.h

Removed: 
    


################################################################################
diff  --git a/clang/include/clang/Basic/LLVM.h b/clang/include/clang/Basic/LLVM.h
index 7ffc4c403473b..f4956cd16cbcf 100644
--- a/clang/include/clang/Basic/LLVM.h
+++ b/clang/include/clang/Basic/LLVM.h
@@ -19,9 +19,6 @@
 // dependencies.
 // Casting.h has complex templates that cannot be easily forward declared.
 #include "llvm/Support/Casting.h"
-// None.h includes an enumerator that is desired & cannot be forward declared
-// without a definition of NoneType.
-#include "llvm/ADT/None.h"
 // Add this header as a workaround to prevent `too few template arguments for
 // class template 'SmallVector'` building error with build compilers like XL.
 #include "llvm/ADT/SmallVector.h"
@@ -37,7 +34,6 @@ namespace llvm {
   template<unsigned InternalLen> class SmallString;
   template<typename T, unsigned N> class SmallVector;
   template<typename T> class SmallVectorImpl;
-  template <typename T> using Optional = std::optional<T>;
   template <class T> class Expected;
 
   template<typename T>
@@ -69,7 +65,6 @@ namespace clang {
   // ADT's.
   using llvm::ArrayRef;
   using llvm::MutableArrayRef;
-  using llvm::Optional;
   using llvm::OwningArrayRef;
   using llvm::SaveAndRestore;
   using llvm::SmallString;