commit 7c03b80ef26eef1b34775b035b66a2531d917595
author Derek Mauro <dmauro@google.com> Thu Jun 13 09:28:19 2024 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Jun 13 09:29:15 2024 -0700
tree 711b5cab416511f55135cc391711cd6f3536a53b
parent 7c17d8bc53d2761b094faed27a04db2fb8df1243

Use ABSL_UNREACHABLE() instead of equivalent

PiperOrigin-RevId: 643024432
Change-Id: Id07aa18d186291442f7b6f3c68ef8dd6cc20b434

absl/types/internal/variant.h

diff --git a/absl/types/internal/variant.h b/absl/types/internal/variant.h
index 40f57c4..4cb15f2 100644
--- a/absl/types/internal/variant.h
+++ b/absl/types/internal/variant.h
@@ -273,20 +273,7 @@
   template <class Op>
   [[noreturn]] static VisitIndicesResultT<Op, std::size_t> Run(
       Op&& /*ignored*/) {
-#if ABSL_HAVE_BUILTIN(__builtin_unreachable) || \
-    (defined(__GNUC__) && !defined(__clang__))
-    __builtin_unreachable();
-#elif defined(_MSC_VER)
-    __assume(false);
-#else
-    // Try to use assert of false being identified as an unreachable intrinsic.
-    // NOTE: We use assert directly to increase chances of exploiting an assume
-    //       intrinsic.
-    assert(false);  // NOLINT
-
-    // Hack to silence potential no return warning -- cause an infinite loop.
-    return Run(std::forward<Op>(op));
-#endif  // Checks for __builtin_unreachable
+    ABSL_UNREACHABLE();
   }
 };