I must have forgotten to add a doxygen comment on it. Will do today
On Tue, Oct 11, 2016 at 7:09 AM Richard Chamberlain via lldb-commits <
lldb-commits@lists.llvm.org> wrote:
> rnchamberlain added a comment.
>
> Thanks, that's nicer. I did have a look for a 'contains()' method, but it
> has not
rnchamberlain added a comment.
Thanks, that's nicer. I did have a look for a 'contains()' method, but it has
not made it into the docs yet.
http://llvm.org/docs/doxygen/html/classllvm_1_1StringRef.html
https://reviews.llvm.org/D25179
___
rnchamberlain updated this revision to Diff 74248.
rnchamberlain added a comment.
Updating as suggested to use StringRef.contains()
https://reviews.llvm.org/D25179
Files:
source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp
Index: source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp
hhellyer added a comment.
The change seems unlikely to pick up many false positives and core dumps from
the type of system described in the bug are only going to get more common.
https://reviews.llvm.org/D25179
___
lldb-commits mailing list
rnchamberlain added a comment.
More information, including readelf output from the dump, in the bugzilla here:
https://llvm.org/bugs/show_bug.cgi?id=30485
https://reviews.llvm.org/D25179
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
rnchamberlain created this revision.
rnchamberlain added reviewers: clayborg, ted, hhellyer.
rnchamberlain added a subscriber: lldb-commits.
ObjectFileELF::RefineModuleDetailsFromNote() identifies Linux core dumps by
searching for
library paths starting with /lib/x86_64-linux-gnu or