[llvm] 319c119 - [YAML] Init local var not set by some branches (#123137)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 15 16:19:53 PST 2025
Author: Vitaly Buka
Date: 2025-01-15T16:19:51-08:00
New Revision: 319c1190939960d009c88baf0d5fb97a549b7a57
URL: https://github.com/llvm/llvm-project/commit/319c1190939960d009c88baf0d5fb97a549b7a57
DIFF: https://github.com/llvm/llvm-project/commit/319c1190939960d009c88baf0d5fb97a549b7a57.diff
LOG: [YAML] Init local var not set by some branches (#123137)
It will not be set if:
1. `(TypeStr.starts_with("SHT_") || isInteger(TypeStr)) == false`: here
we want go to switch default.
2. `IO.mapRequired("Type", Type);` fail parsing. It sets error
internally, so probably not important what happen next, so it's go to
the switch
Added:
Modified:
llvm/lib/ObjectYAML/ELFYAML.cpp
Removed:
################################################################################
diff --git a/llvm/lib/ObjectYAML/ELFYAML.cpp b/llvm/lib/ObjectYAML/ELFYAML.cpp
index 7e94d01a971534..83e6cf76dd746f 100644
--- a/llvm/lib/ObjectYAML/ELFYAML.cpp
+++ b/llvm/lib/ObjectYAML/ELFYAML.cpp
@@ -1588,7 +1588,7 @@ static bool isInteger(StringRef Val) {
void MappingTraits<std::unique_ptr<ELFYAML::Chunk>>::mapping(
IO &IO, std::unique_ptr<ELFYAML::Chunk> &Section) {
- ELFYAML::ELF_SHT Type;
+ ELFYAML::ELF_SHT Type = ELF::ET_NONE;
StringRef TypeStr;
if (IO.outputting()) {
if (auto *S = dyn_cast<ELFYAML::Section>(Section.get()))
More information about the llvm-commits
mailing list