[llvm] [NFC][LLVM] Namespace cleanup in MetadataLoader.cpp (PR #157595)
Rahul Joshi via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 8 19:59:18 PDT 2025
https://github.com/jurahul created https://github.com/llvm/llvm-project/pull/157595
- Restrict scope of anonymous namespaces to just class declarations.
- Move local static function out of anonymous namespace.
- Remove a redundant assert when indexing a vector
>From 3cba71758de9ec8750416cf148a92b89c6641bc1 Mon Sep 17 00:00:00 2001
From: Rahul Joshi <rjoshi at nvidia.com>
Date: Mon, 8 Sep 2025 19:57:02 -0700
Subject: [PATCH] [NFC][LLVM] Namespace cleanup in MetadataLoader.cpp
- Restrict scope of anonymous namespaces to just class declarations.
- Move local static function out of anonymous namespace.
- Remove a redundant assert when indexing a vector
---
llvm/lib/Bitcode/Reader/MetadataLoader.cpp | 15 ++++-----------
1 file changed, 4 insertions(+), 11 deletions(-)
diff --git a/llvm/lib/Bitcode/Reader/MetadataLoader.cpp b/llvm/lib/Bitcode/Reader/MetadataLoader.cpp
index a5cedadd30981..6cb1868738561 100644
--- a/llvm/lib/Bitcode/Reader/MetadataLoader.cpp
+++ b/llvm/lib/Bitcode/Reader/MetadataLoader.cpp
@@ -58,9 +58,6 @@
#include <tuple>
#include <utility>
#include <vector>
-namespace llvm {
-class Argument;
-}
using namespace llvm;
@@ -83,8 +80,6 @@ static cl::opt<bool> DisableLazyLoading(
namespace {
-static int64_t unrotateSign(uint64_t U) { return (U & 1) ? ~(U >> 1) : U >> 1; }
-
class BitcodeReaderMetadataList {
/// Array of metadata references.
///
@@ -129,10 +124,7 @@ class BitcodeReaderMetadataList {
void pop_back() { MetadataPtrs.pop_back(); }
bool empty() const { return MetadataPtrs.empty(); }
- Metadata *operator[](unsigned i) const {
- assert(i < MetadataPtrs.size());
- return MetadataPtrs[i];
- }
+ Metadata *operator[](unsigned i) const { return MetadataPtrs[i]; }
Metadata *lookup(unsigned I) const {
if (I < MetadataPtrs.size())
@@ -178,6 +170,9 @@ class BitcodeReaderMetadataList {
private:
Metadata *resolveTypeRefArray(Metadata *MaybeTuple);
};
+} // namespace
+
+static int64_t unrotateSign(uint64_t U) { return (U & 1) ? ~(U >> 1) : U >> 1; }
void BitcodeReaderMetadataList::assignValue(Metadata *MD, unsigned Idx) {
if (auto *MDN = dyn_cast<MDNode>(MD))
@@ -392,8 +387,6 @@ void PlaceholderQueue::flush(BitcodeReaderMetadataList &MetadataList) {
}
}
-} // anonymous namespace
-
static Error error(const Twine &Message) {
return make_error<StringError>(
Message, make_error_code(BitcodeError::CorruptedBitcode));
More information about the llvm-commits
mailing list