[PATCH] D78820: [LLD][ELF] Eliminate symbols of merged .ARM.exidx sections.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 01:01:27 PDT 2020


grimar added a comment.

(a minor general comment about the code from me).



================
Comment at: lld/ELF/SyntheticSections.cpp:2034
+    auto *d = dyn_cast<Defined>(s.sym);
+    assert(d);
+    return d->section && !d->section->repl->isLive();
----------------
I think you wanted to use `cast`:
```
auto *d = cast<Defined>(s.sym);
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78820





More information about the llvm-commits mailing list