[llvm] [DebugInfo][ObjectYAML] Remove duplicate "Flags" field from LabelSym (PR #88194)

via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 9 14:00:02 PDT 2024


https://github.com/nikitalita created https://github.com/llvm/llvm-project/pull/88194

There was a duplicate flags field mistakenly left in LabelSym. [LabelSym only has one flags field](https://github.com/microsoft/microsoft-pdb/blob/805655a28bd8198004be2ac27e6e0290121a5e89/include/cvinfo.h#L3806

>From 85c647ff67f1ecc6d23471efd66531938e81c2e3 Mon Sep 17 00:00:00 2001
From: nikitalita <69168929+nikitalita at users.noreply.github.com>
Date: Thu, 2 Nov 2023 05:34:24 -0700
Subject: [PATCH] [DebugInfo][ObjectYAML] Remove duplicate "Flags" field from
 LabelSym

---
 llvm/lib/ObjectYAML/CodeViewYAMLSymbols.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/llvm/lib/ObjectYAML/CodeViewYAMLSymbols.cpp b/llvm/lib/ObjectYAML/CodeViewYAMLSymbols.cpp
index 64e1a58aa71a8a..e1d2700623ec3c 100644
--- a/llvm/lib/ObjectYAML/CodeViewYAMLSymbols.cpp
+++ b/llvm/lib/ObjectYAML/CodeViewYAMLSymbols.cpp
@@ -467,7 +467,6 @@ template <> void SymbolRecordImpl<LabelSym>::map(IO &IO) {
   IO.mapOptional("Offset", Symbol.CodeOffset, 0U);
   IO.mapOptional("Segment", Symbol.Segment, uint16_t(0));
   IO.mapRequired("Flags", Symbol.Flags);
-  IO.mapRequired("Flags", Symbol.Flags);
   IO.mapRequired("DisplayName", Symbol.Name);
 }
 



More information about the llvm-commits mailing list