[PATCH] D58047: [LLD][ELF][ARM] Synthesise missing .ARM.exidx sections.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 11 10:23:11 PST 2019


ruiu added inline comments.


================
Comment at: ELF/SyntheticSections.cpp:3055
+      LinkSec(Link) {
+  IsSentinel = Link == nullptr;
+  RawData = Data;
----------------
You are assigning a `nullptr` to a boolean member.

Who sets IsSentinel to true? Looks like there's no code doing that.


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

https://reviews.llvm.org/D58047





More information about the llvm-commits mailing list