[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