On Fri, May 16, 2014 at 9:55 PM, Eran Ifrah <[email protected]> wrote:
> > > > On Fri, May 16, 2014 at 8:23 PM, Deepak Panickal <[email protected]>wrote: > >> Hi Eran, >> >> I tried it here with a Debug build and a MinSizeRel build, but no crashes. >> >> Did you try to launch it from an IDE or simply ran it? >> > I tried it from the command line > > FYI: > I am using Windows 7 (64), Visual Studio 2013 (express edition) > Can you upload your lldb-mi.exe / liblldb.dll somewhere so I could try it > here and see if its a problem with my build? > > > > >> Could you please try a debug build? >> > Will do > > here is the backtrace from the crash (it crashes on a va_copy() call... arg) : msvcr120d.dll!5ccefb53() Unknown [Frames below may be incorrect and/or missing, no symbols loaded for msvcr120d.dll] [External Code] > lldb-mi.exe!CMIUtilString::FormatPriv(const CMIUtilString & vrFormat, char * vArgs) Line 114 C++ lldb-mi.exe!CMIUtilString::Format(const CMIUtilString & vrFormating, ...) Line 139 C++ lldb-mi.exe!CMIDriverMgr::GetAppVersion() Line 577 C++ lldb-mi.exe!CMICmnLog::Initialize() Line 83 C++ lldb-mi.exe!MI::ModuleInit<CMICmnLog>(const int vErrorResrcId, bool & vwrbOk, CMIUtilString & vwrErrMsg) Line 50 C++ lldb-mi.exe!CMIDriverMgr::Initialize() Line 82 C++ lldb-mi.exe!DriverSystemInit() Line 199 C++ lldb-mi.exe!main(int argc, const char * * argv) Line 317 C++ [External Code] And for your convenience, here is the "Locals" view during the crash: vrFormat "%s\n%s\n%s" const CMIUtilString & vArgs 0x00bdf334 "ÌÌÌÌ| \x15" char * argsCpy 0x00bdf334 "ÌÌÌÌ| \x15" char * nFinal 0 int pFormatted unique_ptr 37 '%' std::unique_ptr<char [0],std::default_delete<char [0]> > n 16 int strResult "" CMIUtilString Hope it will give you a hint > >> We'll look into it more to see if we can find something. >> >> Thanks, >> Deepak >> >> >> On 16/05/2014 17:04, Eran Ifrah wrote: >> >> Hi Deepak, >> >> I have just updated and rebuilt the tree on Windows >> Starting lldb-mi.exe -> instantly crashes >> for comparison, starting lldb.exe it starts normally without any crash >> >> I built it using VS2013, I selected the 'MinSizeRel' build >> configuration >> Unfortunately, running it under VS, gives a useless backtrace (only >> addresses) >> >> I could try and build it with debug info, but it just takes too much >> time :D >> >> Eran >> >> >> >> >> On Fri, May 16, 2014 at 6:50 PM, <[email protected]> wrote: >> >>> Excellent! Again, thanks for working on this. >>> >>> Jim >>> >>> On May 16, 2014, at 4:15 AM, Deepak Panickal <[email protected]> >>> wrote: >>> >>> > Thanks Jim, have just committed it. Yeah, it works with the GDB/MI >>> interface. >>> > >>> > There's no GPL code, it's all written from scratch. >>> > Not all MI commands have been implemented yet, however it should be >>> enough to set breakpoints, view locals. >>> > We're working on adding more commands for viewing/modifying registers, >>> memory etc. >>> > >>> > It'll be great if more people from the community can use and test it >>> with other GUIs! >>> > >>> > Thanks, >>> > Deepak >>> > >>> > >>> > On 15/05/2014 18:57, [email protected] wrote: >>> >> First off, thanks for doing this, that is great! If this also works >>> with the gdb-mi mode, you will make our emacs friends happy as well. >>> >> >>> >> If this is just an independent tool that doesn't touch the core lldb >>> sources (and doesn't use anything directly from the gdb MI implementation, >>> since we don't want to mix GPL'ed code in with lldb), I see no reason not >>> to just check it in now, and then iterate on it in situ. That is much >>> easier than handing around patch files, and since there's no chance it will >>> interfere with anybody working on parts of lldb that existed before the >>> patch, can't do any harm. I spoke to GregC and he agrees this is a >>> reasonable course. >>> >> >>> >> Again, thanks for working on this. >>> >> >>> >> Jim >>> >> >>> >> >>> >> On May 15, 2014, at 10:17 AM, Deepak Panickal <[email protected]> >>> wrote: >>> >> >>> >>> Hi Abid, >>> >>> >>> >>> Thank you, I have attached the zipped patch. >>> >>> You might have to the change the triple in >>> tools/lldb-mi/MICmnConfig.h for your target. >>> >>> >>> >>> Please let me know if you have any comments/suggestions. >>> >>> >>> >>> Thanks, >>> >>> Deepak >>> >>> >>> >>> On 15/05/2014 17:10, Abid, Hafiz wrote: >>> >>>> Hi Deepak, >>> >>>> I can help review/test it when you post the patch. >>> >>>> >>> >>>> Thanks, >>> >>>> Abid >>> >>>> >>> >>>>> -----Original Message----- >>> >>>>> From: [email protected] [mailto: >>> [email protected]] >>> >>>>> On Behalf Of Deepak Panickal >>> >>>>> Sent: 15 May 2014 12:14 >>> >>>>> To: [email protected] >>> >>>>> Subject: [lldb-dev] LLDB Machine Interface Frontend >>> >>>>> >>> >>>>> Hi, >>> >>>>> >>> >>>>> We have an MI (Machine Interface) frontend to LLDB which works out >>> of the >>> >>>>> box with Eclipse, and likely will work with other GUI's based on >>> the GDB/MI >>> >>>>> interface. It's been tested on Linux and Windows. >>> >>>>> >>> >>>>> It's ready to upstream and has been developed as an independent >>> tool >>> >>>>> residing in the LLDB tools folder. It's based on the LLDB public >>> API without >>> >>>>> any changes to LLDB internals. >>> >>>>> >>> >>>>> The patch is quite large as there are many new files. >>> >>>>> How should we go about upstreaming? Should I create a diff review >>> as >>> >>>>> usual? >>> >>>>> >>> >>>>> Thanks, >>> >>>>> Deepak >>> >>>>> _______________________________________________ >>> >>>>> lldb-dev mailing list >>> >>>>> [email protected] >>> >>>>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev >>> >>> >>> <lldb-mi-driver.patch.bz2>_______________________________________________ >>> >>> lldb-dev mailing list >>> >>> [email protected] >>> >>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev >>> > >>> >>> _______________________________________________ >>> lldb-dev mailing list >>> [email protected] >>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev >>> >> >> >> >> -- >> Eran Ifrah >> Author of codelite, a cross platform open source C/C++ IDE: >> http://www.codelite.org >> wxCrafter, a wxWidgets RAD: http://wxcrafter.codelite.org >> >> >> > > > -- > Eran Ifrah > Author of codelite, a cross platform open source C/C++ IDE: > http://www.codelite.org > wxCrafter, a wxWidgets RAD: http://wxcrafter.codelite.org > -- Eran Ifrah Author of codelite, a cross platform open source C/C++ IDE: http://www.codelite.org wxCrafter, a wxWidgets RAD: http://wxcrafter.codelite.org
_______________________________________________ lldb-dev mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
