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

via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 11 03:08:41 PDT 2024


Author: nikitalita
Date: 2024-04-11T11:08:38+01:00
New Revision: cca30dfb5935e05837e37cced4407a63393c6642

URL: https://github.com/llvm/llvm-project/commit/cca30dfb5935e05837e37cced4407a63393c6642
DIFF: https://github.com/llvm/llvm-project/commit/cca30dfb5935e05837e37cced4407a63393c6642.diff

LOG: [DebugInfo][ObjectYAML] Remove duplicate "Flags" field from LabelSym (#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)

Added: 
    

Modified: 
    llvm/lib/ObjectYAML/CodeViewYAMLSymbols.cpp

Removed: 
    


################################################################################
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