summaryrefslogtreecommitdiffstats
path: root/llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp
diff options
context:
space:
mode:
authorGreg Clayton <gclayton@apple.com>2014-11-03 21:02:54 +0000
committerGreg Clayton <gclayton@apple.com>2014-11-03 21:02:54 +0000
commit118593a3af18af46be14cefbb5e5f2d20ea9471b (patch)
tree4add80a991159e21f22264e8201d40f86e5768e2 /llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp
parentb8ceaf15cdf7eacc63630d024f0ceeae9d85c722 (diff)
downloadbcm5719-llvm-118593a3af18af46be14cefbb5e5f2d20ea9471b.tar.gz
bcm5719-llvm-118593a3af18af46be14cefbb5e5f2d20ea9471b.zip
The change previously committed as 220983 broke large binary memory reads. I kept the "idx - 1" fix from 220983, but reverted the while loop that was incorrectly added.
The details are: large packets (like large memory reads (m packets) or large binary memory reads (x packet)) can get responses that come in across multiple read() calls. The while loop that was added meant that if only a partial packet came in (like only "$abc" coming for a response) GDBRemoteCommunication::CheckForPacket() was called, it would deadlock in the while loop because no more data is going to come in as this function needs to be called again with more data from another read. So the original fix will need to be corrected and resubmitted. <rdar://problem/18853744> llvm-svn: 221181
Diffstat (limited to 'llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud