Author: jmolenda
Date: Tue Jan 24 00:09:06 2017
New Revision: 292890

URL: http://llvm.org/viewvc/llvm-project?rev=292890&view=rev
Log:
Fix the last commit; compression was being enabled on mac native
which led to

ERROR: test_auxv_chunked_reads_work_debugserver 
(tools/lldb-server/TestGdbRemoteAuxvSupport.py)
ERROR: test_auxv_data_is_correct_size_debugserver 
(tools/lldb-server/TestGdbRemoteAuxvSupport.py)
ERROR: test_auxv_keys_look_valid_debugserver 
(tools/lldb-server/TestGdbRemoteAuxvSupport.py)
ERROR: test_qSupported_returns_known_stub_features_debugserver 
(tools/lldb-server/TestLldbGdbServer.py)

failures because debugserver was advertising compression being available, e.g.

send packet: $qSupported:xmlRegisters=i386,arm,mips#12
read packet: 
$qXfer:features:read+;PacketSize=20000;qEcho+;SupportedCompressions=zlib-deflate;DefaultCompressionMinSize=384#00

maybe these tests should be a little more accepting of additional
features.  but I didn't mean for this to be enabled on mac native.

Modified:
    lldb/trunk/tools/debugserver/source/RNBRemote.cpp

Modified: lldb/trunk/tools/debugserver/source/RNBRemote.cpp
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/RNBRemote.cpp?rev=292890&r1=292889&r2=292890&view=diff
==============================================================================
--- lldb/trunk/tools/debugserver/source/RNBRemote.cpp (original)
+++ lldb/trunk/tools/debugserver/source/RNBRemote.cpp Tue Jan 24 00:09:06 2017
@@ -3614,7 +3614,7 @@ rnb_err_t RNBRemote::HandlePacket_qSuppo
   bool enable_compression = false;
   (void)enable_compression;
 
-#if defined (TARGET_OS_WATCH) || defined (TARGET_OS_IOS) || defined 
(TARGET_OS_TVOS)
+#if (defined (TARGET_OS_WATCH) && TARGET_OS_WATCHOS == 1) || (defined 
(TARGET_OS_IOS) && TARGET_OS_IOS == 1) || (defined (TARGET_OS_TVOS) && 
TARGET_OS_TVOS == 1)
   enable_compression = true;
 #endif
 


_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to