[PATCH] D79560: [Object] Remove unneeded check in ELFFile<ELFT>::dynamicEntries().
Xing GUO via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun May 10 22:27:34 PDT 2020
Higuoxing marked 2 inline comments as done.
Higuoxing added inline comments.
================
Comment at: llvm/lib/Object/ELF.cpp:505
ArrayRef<Elf_Dyn> Dyn;
size_t DynSecSize = 0;
----------------
abhinavgaba wrote:
> This is not being read anymore. So it's causing these warnings:
> `variable ‘DynSecSize’ set but not used [-Werror=unused-but-set-variable]`.
Fixed in rG1171bef0209de3367ad3837745dbc19e9d0cf1ed, thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79560/new/
https://reviews.llvm.org/D79560
More information about the llvm-commits
mailing list