[PATCH] D71857: [NFC] Fixes -Wrange-loop-analysis warnings

Mark de Wever via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 4 11:20:31 PST 2020


Mordante added a comment.

In D71857#1800663 <https://reviews.llvm.org/D71857#1800663>, @MaskRay wrote:

> I think there is a false positive.
>
> https://github.com/llvm/llvm-project/tree/master/lld/ELF/Relocations.cpp#L1622
>
>   for (const std::pair<ThunkSection *, uint32_t> ts : isd->thunkSections)
>   


Removing the `const` like L1648 will also solve the issue.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71857/new/

https://reviews.llvm.org/D71857





More information about the cfe-commits mailing list