commit 5aa0cff7e964df5e9d3fb634706407a58ec45575
author Fangrui Song <i@maskray.me> Fri Mar 01 11:17:22 2024 -0800
committer Copybara-Service <copybara-worker@google.com> Fri Mar 01 11:23:39 2024 -0800
tree d2d8f027b61806f609b6d07800b3e84c016759d7
parent 89eb5667dda3f4529eb779ad5d42f4bb86149162

[ELF] Internalize enum

g++ -flto has a diagnostic `-Wodr` about mismatched redeclarations,
which even apply to `enum`.

Fix #83529

Reviewers: thesamesam

Reviewed By: thesamesam

Pull Request: https://github.com/llvm/llvm-project/pull/83604

GitOrigin-RevId: 4a3f7e798a31072a80a0731b8fb1da21b9c626ed

ELF/Arch/LoongArch.cpp

diff --git a/ELF/Arch/LoongArch.cpp b/ELF/Arch/LoongArch.cpp
index 49fd979..e0f9abf 100644
--- a/ELF/Arch/LoongArch.cpp
+++ b/ELF/Arch/LoongArch.cpp
@@ -41,6 +41,7 @@
 };
 } // end anonymous namespace
 
+namespace {
 enum Op {
   SUB_W = 0x00110000,
   SUB_D = 0x00118000,
@@ -65,6 +66,7 @@
   R_T2 = 14,
   R_T3 = 15,
 };
+} // namespace
 
 // Mask out the input's lowest 12 bits for use with `pcalau12i`, in sequences
 // like `pcalau12i + addi.[wd]` or `pcalau12i + {ld,st}.*` where the `pcalau12i`

ELF/Arch/PPC64.cpp

diff --git a/ELF/Arch/PPC64.cpp b/ELF/Arch/PPC64.cpp
index de52f6a..019c073 100644
--- a/ELF/Arch/PPC64.cpp
+++ b/ELF/Arch/PPC64.cpp
@@ -26,6 +26,7 @@
 constexpr uint64_t ppc64TocOffset = 0x8000;
 constexpr uint64_t dynamicThreadPointerOffset = 0x8000;
 
+namespace {
 // The instruction encoding of bits 21-30 from the ISA for the Xform and Dform
 // instructions that can be used as part of the initial exec TLS sequence.
 enum XFormOpcd {
@@ -139,6 +140,7 @@
   PSTXV = PREFIX_8LS | 0xd8000000,
   PSTXVP = PREFIX_8LS | 0xf8000000
 };
+
 static bool checkPPCLegacyInsn(uint32_t encoding) {
   PPCLegacyInsn insn = static_cast<PPCLegacyInsn>(encoding);
   if (insn == PPCLegacyInsn::NOINSN)
@@ -164,7 +166,6 @@
       0x8000000003e00000, // S/T (6-10) - The [S/T]X bit moves from 28 to 5.
 };
 
-namespace {
 class PPC64 final : public TargetInfo {
 public:
   PPC64();

ELF/Arch/RISCV.cpp

diff --git a/ELF/Arch/RISCV.cpp b/ELF/Arch/RISCV.cpp
index 5fcab4d..4798c86 100644
--- a/ELF/Arch/RISCV.cpp
+++ b/ELF/Arch/RISCV.cpp
@@ -57,6 +57,7 @@
 
 const uint64_t dtpOffset = 0x800;
 
+namespace {
 enum Op {
   ADDI = 0x13,
   AUIPC = 0x17,
@@ -78,6 +79,7 @@
   X_A0 = 10,
   X_T3 = 28,
 };
+} // namespace
 
 static uint32_t hi20(uint32_t val) { return (val + 0x800) >> 12; }
 static uint32_t lo12(uint32_t val) { return val & 4095; }