commit b5f8d42efe3e246d582d4a1a328fac915e4ce8dc
author Kazu Hirata <kazu@google.com> Sun Aug 07 11:30:58 2022 -0700
committer Kazu Hirata <kazu@google.com> Sun Aug 07 11:30:58 2022 -0700
tree 4f38480244b4641272146bf8c1d07f59b8b28923
parent aa2a3196e28425c6bcf74bbf913efeda1f7fa1c7

[ADT] Deprecate Optional::{hasValue,getValue} (NFC)

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

llvm/include/llvm/ADT/Optional.h

diff --git a/llvm/include/llvm/ADT/Optional.h b/llvm/include/llvm/ADT/Optional.h
index e164dc0..4f9e098 100644
--- a/llvm/include/llvm/ADT/Optional.h
+++ b/llvm/include/llvm/ADT/Optional.h
@@ -92,13 +92,16 @@
   }
 
   constexpr bool has_value() const noexcept { return hasVal; }
-  constexpr bool hasValue() const noexcept { return hasVal; }
+  [[deprecated("Use has_value instead.")]] constexpr bool
+  hasValue() const noexcept {
+    return hasVal;
+  }
 
   T &value() &noexcept {
     assert(hasVal);
     return val;
   }
-  T &getValue() &noexcept {
+  [[deprecated("Use value instead.")]] T &getValue() &noexcept {
     assert(hasVal);
     return val;
   }
@@ -106,7 +109,8 @@
     assert(hasVal);
     return val;
   }
-  constexpr T const &getValue() const &noexcept {
+  [[deprecated("Use value instead.")]] constexpr T const &
+  getValue() const &noexcept {
     assert(hasVal);
     return val;
   }
@@ -114,7 +118,7 @@
     assert(hasVal);
     return std::move(val);
   }
-  T &&getValue() &&noexcept {
+  [[deprecated("Use value instead.")]] T &&getValue() &&noexcept {
     assert(hasVal);
     return std::move(val);
   }
@@ -203,13 +207,16 @@
   }
 
   constexpr bool has_value() const noexcept { return hasVal; }
-  constexpr bool hasValue() const noexcept { return hasVal; }
+  [[deprecated("Use has_value instead.")]] constexpr bool
+  hasValue() const noexcept {
+    return hasVal;
+  }
 
   T &value() &noexcept {
     assert(hasVal);
     return val;
   }
-  T &getValue() &noexcept {
+  [[deprecated("Use value instead.")]] T &getValue() &noexcept {
     assert(hasVal);
     return val;
   }
@@ -217,7 +224,8 @@
     assert(hasVal);
     return val;
   }
-  constexpr T const &getValue() const &noexcept {
+  [[deprecated("Use value instead.")]] constexpr T const &
+  getValue() const &noexcept {
     assert(hasVal);
     return val;
   }
@@ -225,7 +233,7 @@
     assert(hasVal);
     return std::move(val);
   }
-  T &&getValue() &&noexcept {
+  [[deprecated("Use value instead.")]] T &&getValue() &&noexcept {
     assert(hasVal);
     return std::move(val);
   }
@@ -303,13 +311,19 @@
   constexpr const T *getPointer() const { return &Storage.value(); }
   T *getPointer() { return &Storage.value(); }
   constexpr const T &value() const & { return Storage.value(); }
-  constexpr const T &getValue() const & { return Storage.value(); }
+  [[deprecated("Use value instead.")]] constexpr const T &getValue() const & {
+    return Storage.value();
+  }
   T &value() & { return Storage.value(); }
-  T &getValue() & { return Storage.value(); }
+  [[deprecated("Use value instead.")]] T &getValue() & {
+    return Storage.value();
+  }
 
   constexpr explicit operator bool() const { return has_value(); }
   constexpr bool has_value() const { return Storage.has_value(); }
-  constexpr bool hasValue() const { return Storage.has_value(); }
+  [[deprecated("Use has_value instead.")]] constexpr bool hasValue() const {
+    return Storage.has_value();
+  }
   constexpr const T *operator->() const { return getPointer(); }
   T *operator->() { return getPointer(); }
   constexpr const T &operator*() const & { return value(); }
@@ -333,7 +347,9 @@
   }
 
   T &&value() && { return std::move(Storage.value()); }
-  T &&getValue() && { return std::move(Storage.value()); }
+  [[deprecated("Use value instead.")]] T &&getValue() && {
+    return std::move(Storage.value());
+  }
   T &&operator*() && { return std::move(Storage.value()); }
 
   template <typename U> T value_or(U &&alt) && {