[llvm] [CAS] OnDiskGraphDB - fix MSVC "not all control paths return a value" warnings. NFC. (PR #164369)

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 21 00:28:11 PDT 2025


https://github.com/RKSimon created https://github.com/llvm/llvm-project/pull/164369

None

>From 342fcf41e060ea43637568b1741c09febe7598b5 Mon Sep 17 00:00:00 2001
From: Simon Pilgrim <llvm-dev at redking.me.uk>
Date: Tue, 21 Oct 2025 08:26:46 +0100
Subject: [PATCH] [CAS] OnDiskGraphDB - fix MSVC "not all control paths return
 a value" warnings. NFC.

---
 llvm/include/llvm/CAS/OnDiskGraphDB.h | 1 +
 llvm/lib/CAS/OnDiskGraphDB.cpp        | 3 +++
 2 files changed, 4 insertions(+)

diff --git a/llvm/include/llvm/CAS/OnDiskGraphDB.h b/llvm/include/llvm/CAS/OnDiskGraphDB.h
index 83017a6a54fee..5f0ee0e131c0f 100644
--- a/llvm/include/llvm/CAS/OnDiskGraphDB.h
+++ b/llvm/include/llvm/CAS/OnDiskGraphDB.h
@@ -380,6 +380,7 @@ class OnDiskGraphDB {
     case ObjectPresence::OnlyInUpstreamDB:
       return true;
     }
+    llvm_unreachable("Unknown ObjectPresence enum");
   }
 
   /// When \p load is called for a node that doesn't exist, this function tries
diff --git a/llvm/lib/CAS/OnDiskGraphDB.cpp b/llvm/lib/CAS/OnDiskGraphDB.cpp
index 72bb98c4bf65d..64cbe9dc8e159 100644
--- a/llvm/lib/CAS/OnDiskGraphDB.cpp
+++ b/llvm/lib/CAS/OnDiskGraphDB.cpp
@@ -836,6 +836,7 @@ uint64_t DataRecordHandle::getDataSize() const {
   case DataSizeFlags::Uses8B:
     return support::endian::read64le(DataSizePtr);
   }
+  llvm_unreachable("Unknown DataSizeFlags enum");
 }
 
 void DataRecordHandle::skipDataSize(LayoutFlags LF, int64_t &RelOffset) const {
@@ -863,6 +864,7 @@ uint32_t DataRecordHandle::getNumRefs() const {
   case NumRefsFlags::Uses8B:
     return support::endian::read64le(NumRefsPtr);
   }
+  llvm_unreachable("Unknown NumRefsFlags enum");
 }
 
 void DataRecordHandle::skipNumRefs(LayoutFlags LF, int64_t &RelOffset) const {
@@ -1270,6 +1272,7 @@ Expected<bool> OnDiskGraphDB::isMaterialized(ObjectID Ref) {
       return FaultInResult.takeError();
     return true;
   }
+  llvm_unreachable("Unknown ObjectPresence enum");
 }
 
 Expected<OnDiskGraphDB::ObjectPresence>



More information about the llvm-commits mailing list