commit | 8c67829e508974911b9c657a29e1d051387fef54 | [log] [tgz] |
---|---|---|
author | Derek Cheng <imcheng@chromium.org> | Tue Dec 05 23:40:26 2017 |
committer | Commit Bot <commit-bot@chromium.org> | Tue Dec 05 23:40:26 2017 |
tree | f8d47c76e25aa073b3becceab0cd4b9c6ea85b54 | |
parent | a761eb2c72c3053897a0658dfb784cc9aad8eaf7 [diff] |
Revert "Use llvm symbolizer for stack script." This reverts commit b6e038a7a7b07f680eba6d6ec1c32319ae89cd78. Reason for revert: Broke stack_tool_with_logcat_dump on WebKit Android (Nexus4): https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/72002 Original change's description: > Use llvm symbolizer for stack script. > > llvm symbolizer is a more efficient symbolizer than addr2line, objdump, > etc. > > In this cl, I 1)created a wrapper instance to interact with > llvm symbolizer, 2)made the stack script to use llvm symbolizer instance, > and then 3)added llvm symbolizer into isolated inputs. > > Bug: 774267 > Change-Id: I971fb808b97f3a569eb9615f99efa41e3a56f3cb > Reviewed-on: https://chromium-review.googlesource.com/789376 > Reviewed-by: agrieve <agrieve@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Commit-Queue: Zhiling Huang <hzl@chromium.org> > Cr-Commit-Position: refs/heads/master@{#521820} TBR=dpranke@chromium.org,agrieve@chromium.org,hzl@chromium.org,bpastene@chromium.org,jbudorick@chromium.org Change-Id: Ieebbcb9527dafd25a0fc74d2826a31162495c86e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 774267 Reviewed-on: https://chromium-review.googlesource.com/809941 Reviewed-by: Derek Cheng <imcheng@chromium.org> Commit-Queue: Derek Cheng <imcheng@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#521898} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 5b391ee57fc58d4beae41346195ed4e1d0fca853