[PATCH] D39749: Allow yaml2obj to order implicit sections for ELF

Dave Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 14:34:28 PST 2017


kastiglione added inline comments.


================
Comment at: llvm/trunk/tools/yaml2obj/yaml2elf.cpp:79-81
+    unsigned Idx;
+    assert(!lookup(Name, Idx) && "Expected section not found in index");
+    return Idx;
----------------
kastiglione wrote:
> The uninitialized `Idx` broke some builds. What's the expedient way to fix this, another diff?
Addressed in r317626.


Repository:
  rL LLVM

https://reviews.llvm.org/D39749





More information about the llvm-commits mailing list