maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[DebugInfoPDB] A few cleanups on PDB Variant class. 

Also implements the PDBSymbolCompilandEnv::getValue() method,
which until now had been unimplemented specifically because
variant did not support string values.

llvm-svn: 261173
This commit is contained in:
Zachary Turner 2016-02-17 22:46:33 +00:00
parent 22d2878399
commit 2a9ac0d2c5
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
llvm/include/llvm/DebugInfo/PDB/PDBTypes.h
View File

@ -363,7 +363,7 @@ struct Variant {
}
~Variant() {
if (Type == PDB_VariantType::String && Value.String != nullptr)
if (Type == PDB_VariantType::String)
delete[] Value.String;
}
@ -410,7 +410,7 @@ struct Variant {
Variant &operator=(const Variant &Other) {
if (this == &Other)
return *this;
if (Type == PDB_VariantType::String && Value.String != nullptr)
if (Type == PDB_VariantType::String)
delete[] Value.String;
Type = Other.Type;
Value = Other.Value;

6
llvm/lib/DebugInfo/PDB/PDBSymbolCompilandEnv.cpp
View File

@ -22,8 +22,10 @@ PDBSymbolCompilandEnv::PDBSymbolCompilandEnv(
: PDBSymbol(PDBSession, std::move(Symbol)) {}
std::string PDBSymbolCompilandEnv::getValue() const {
// call RawSymbol->getValue() and convert the result to an std::string.
return std::string();
llvm::Variant Value = RawSymbol->getValue();
if (Value.Type != PDB_VariantType::String)
return std::string();
return std::string(Value.Value.String);
}
void PDBSymbolCompilandEnv::dump(PDBSymDumper &Dumper) const {