From 22b36bfa3f18ef1cc858d3301c4eea00a280403a Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu@google.com>
Date: Wed, 26 Jun 2024 12:05:58 -0700
Subject: [PATCH] [Memprof] Fix a warning

This patch fixes:

  llvm/lib/ProfileData/MemProfReader.cpp:685:1: error: non-void
  function does not return a value in all con trol paths
  [-Werror,-Wreturn-type]

While I am at it, this patch removes an else-after-return.
---
 llvm/lib/ProfileData/MemProfReader.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/llvm/lib/ProfileData/MemProfReader.cpp b/llvm/lib/ProfileData/MemProfReader.cpp
index 9112e19982ec..85327273d6d7 100644
--- a/llvm/lib/ProfileData/MemProfReader.cpp
+++ b/llvm/lib/ProfileData/MemProfReader.cpp
@@ -677,11 +677,10 @@ llvm::SmallVector<std::pair<uint64_t, MemInfoBlock>>
 RawMemProfReader::readMemInfoBlocks(const char *Ptr) {
   if (MemprofRawVersion == 3ULL)
     return readMemInfoBlocksV3(Ptr);
-  else if (MemprofRawVersion == 4ULL)
+  if (MemprofRawVersion == 4ULL)
     return readMemInfoBlocksV4(Ptr);
-  else
-    assert(false &&
-           "Panic: Unsupported version number when reading MemInfoBlocks");
+  llvm_unreachable(
+      "Panic: Unsupported version number when reading MemInfoBlocks");
 }
 
 Error RawMemProfReader::readRawProfile(