diff options
| author | Christopher Ferris <cferris@google.com> | 2018-03-06 14:02:55 -0800 |
|---|---|---|
| committer | Christopher Ferris <cferris@google.com> | 2018-03-06 16:07:23 -0800 |
| commit | 453e0e557d608d82012a49fedd1b9c83bb54daee (patch) | |
| tree | a6179be23a0ce0ccd053e9398c933cb6634949b4 /compiler/optimizing/graph_visualizer.cc | |
| parent | 7a02c66fd9ed174fc2e49ccc9f582dd661b7de9e (diff) | |
Change Addr2line to use poll.
Moving from select to poll avoids any problems with the fd being
too large for the call to FD_SET. It also slightly simplifies the code.
Make sure that the call to read only ever occurs if there is really data
to be read. There was a case with the previous use of the select call
where the read would be called when there was no data to be read which
caused the process to hang forever.
Test: Ran the 137 test forcing a crash and verified the addr2line
Test: information is correct.
Test: Ran the 141 test forcing a crash and verifying that when more than
Test: two lines are returned by addr2line, it still display them all.
Test: Modified the Addr2line call to pass in an emptry string for the
Test: executable and verified there are no hangs.
Change-Id: I3d750e7796d0c70122686aad1eb40856157c9770
Diffstat (limited to 'compiler/optimizing/graph_visualizer.cc')
0 files changed, 0 insertions, 0 deletions