Hi Eran,

Here's the link to download the MinSizeRel build.
https://db.tt/CdvFKVcN

Thanks,
Deepak


On 16/05/2014 21:01, Eran Ifrah wrote:



On Fri, May 16, 2014 at 9:55 PM, Eran Ifrah <[email protected] <mailto:[email protected]>> wrote:




    On Fri, May 16, 2014 at 8:23 PM, Deepak Panickal
    <[email protected] <mailto:[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]
        <mailto:[email protected]>> wrote:

            Excellent!  Again, thanks for working on this.

            Jim

            On May 16, 2014, at 4:15 AM, Deepak Panickal
            <[email protected] <mailto:[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]
            <mailto:[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] <mailto:[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]>
            [mailto:[email protected]
            <mailto:[email protected]>]
            >>>>> On Behalf Of Deepak Panickal
            >>>>> Sent: 15 May 2014 12:14
            >>>>> To: [email protected] <mailto:[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] <mailto:[email protected]>
            >>>>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
            >>>
            
<lldb-mi-driver.patch.bz2>_______________________________________________
            >>> lldb-dev mailing list
            >>> [email protected] <mailto:[email protected]>
            >>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
            >

            _______________________________________________
            lldb-dev mailing list
            [email protected] <mailto:[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
        <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
    <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 <http://wxcrafter.codelite.org>

_______________________________________________
lldb-dev mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev

Reply via email to