[llvm] 3700587 - [BinaryFormat] Remove redundant declarations (NFC) (#166316)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 4 07:12:41 PST 2025
Author: Kazu Hirata
Date: 2025-11-04T07:12:37-08:00
New Revision: 370058777be2282fe18d62315adafdda3960d012
URL: https://github.com/llvm/llvm-project/commit/370058777be2282fe18d62315adafdda3960d012
DIFF: https://github.com/llvm/llvm-project/commit/370058777be2282fe18d62315adafdda3960d012.diff
LOG: [BinaryFormat] Remove redundant declarations (NFC) (#166316)
In C++17, static constexpr members are implicitly inline, so they no
longer require an out-of-line definition.
Once we remove the redundant declarations, Minidump.cpp becomes
effectively empty, so this patch removes the file.
Identified with readability-redundant-declaration.
Added:
Modified:
llvm/lib/BinaryFormat/CMakeLists.txt
Removed:
llvm/lib/BinaryFormat/Minidump.cpp
################################################################################
diff --git a/llvm/lib/BinaryFormat/CMakeLists.txt b/llvm/lib/BinaryFormat/CMakeLists.txt
index 4b2debb7ae236..0c8af1e7a4565 100644
--- a/llvm/lib/BinaryFormat/CMakeLists.txt
+++ b/llvm/lib/BinaryFormat/CMakeLists.txt
@@ -6,7 +6,6 @@ add_llvm_component_library(LLVMBinaryFormat
ELF.cpp
MachO.cpp
Magic.cpp
- Minidump.cpp
MsgPackDocument.cpp
MsgPackDocumentYAML.cpp
MsgPackReader.cpp
diff --git a/llvm/lib/BinaryFormat/Minidump.cpp b/llvm/lib/BinaryFormat/Minidump.cpp
deleted file mode 100644
index b618fb1570126..0000000000000
--- a/llvm/lib/BinaryFormat/Minidump.cpp
+++ /dev/null
@@ -1,14 +0,0 @@
-//===-- Minidump.cpp - Minidump constants and structures ---------*- C++-*-===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-
-#include "llvm/BinaryFormat/Minidump.h"
-
-using namespace llvm::minidump;
-
-constexpr uint32_t Header::MagicSignature;
-constexpr uint16_t Header::MagicVersion;
More information about the llvm-commits
mailing list