diff --git a/lldb/tools/lldb-mi/MICmdCmdFile.cpp b/lldb/tools/lldb-mi/MICmdCmdFile.cpp index 83e71aae6258..ec415b88e989 100644 --- a/lldb/tools/lldb-mi/MICmdCmdFile.cpp +++ b/lldb/tools/lldb-mi/MICmdCmdFile.cpp @@ -109,10 +109,9 @@ bool CMICmdCmdFileExecAndSymbols::Execute( void ) const CMIUtilString & strExeFilePath( pArgFile->GetValue() ); CMICmnLLDBDebugSessionInfo & rSessionInfo( CMICmnLLDBDebugSessionInfo::Instance() ); lldb::SBDebugger & rDbgr = rSessionInfo.m_rLldbDebugger; - // Developer note: - lldb::SBError error; // Arg options may need to be available here for the CreateTarget() - const char * pTargetTriple = MICONFIG_TRIPLE; // Match the Arm executeable built for the target platform i.e. hello.elf - const char * pTargetPlatformName = ""; // This works for connecting to an Android development board + lldb::SBError error; + const char * pTargetTriple = nullptr; // Let LLDB discover the triple required + const char * pTargetPlatformName = ""; const bool bAddDepModules = false; lldb::SBTarget target = rDbgr.CreateTarget( strExeFilePath.c_str(), pTargetTriple, pTargetPlatformName, bAddDepModules, error ); CMIUtilString strWkDir; @@ -128,6 +127,7 @@ bool CMICmdCmdFileExecAndSymbols::Execute( void ) } if( !rDbgr.SetCurrentPlatformSDKRoot( strWkDir.c_str() ) ) { + SetError( CMIUtilString::Format( MIRSRC( IDS_CMD_ERR_FNFAILED ), m_cmdData.strMiCmd.c_str(), "SetCurrentPlatformSDKRoot()" ) ); return MIstatus::failure; } @@ -178,4 +178,4 @@ bool CMICmdCmdFileExecAndSymbols::Acknowledge( void ) CMICmdBase * CMICmdCmdFileExecAndSymbols::CreateSelf( void ) { return new CMICmdCmdFileExecAndSymbols(); -} \ No newline at end of file +} diff --git a/lldb/tools/lldb-mi/MICmnConfig.h b/lldb/tools/lldb-mi/MICmnConfig.h index 26f1e4219698..6153aacdcf93 100644 --- a/lldb/tools/lldb-mi/MICmnConfig.h +++ b/lldb/tools/lldb-mi/MICmnConfig.h @@ -28,6 +28,7 @@ #define MICONFIG_DEBUG_SHOW_ATTACH_DBG_DLG 0 // 1 = Compile in and init LLDB driver code alongside MI version, 0 = do not use +// ToDo: This has not been fully implemented as may not be required in the future #define MICONFIG_COMPILE_MIDRIVER_WITH_LLDBDRIVER 0 // 1 = Give runtime our own custom buffer, 0 = Use runtime managed buffer @@ -40,8 +41,8 @@ #define MICONFIG_POLL_FOR_STD_IN 1 // Temp workaround while needing different triples -// ToDo: This should be fixed to be figured out automatically -#define MICONFIG_TRIPLE "arm" +// ToDo: Temp workaround while needing different triples - not used ATM, may not be required anymore +//#define MICONFIG_TRIPLE "arm" // 1 = Write to MI's Log file warnings about commands that did not handle arguments or // options present to them by the driver's client, 0 = no warnings given diff --git a/lldb/tools/lldb-mi/MIReadMe.txt b/lldb/tools/lldb-mi/MIReadMe.txt index 316c91236ff1..78a92b513e15 100644 --- a/lldb/tools/lldb-mi/MIReadMe.txt +++ b/lldb/tools/lldb-mi/MIReadMe.txt @@ -84,7 +84,8 @@ CMICmdCmdVarUpdate ========================================================================= MI build configuration: MICmnConfig.h defines various preprocessor build options i.e. enable -LLDB fall through should MI interpretor not recognise a command. +LLDB fall through should MI interpretor not recognise a command (option +not fully implemented - may be removed in the future). ========================================================================= MI uses the following libraries: