[Lldb-commits] [lldb] r296925 - Fix Darwin failures introduced in r296909

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Fri Mar 3 15:48:39 PST 2017


Yea, it was a static method of DataExtractor before.  I noticed the same
thing when i saw it wasn't using any methods of DataExtractor.  Oops!
Anyway since this is no less weird than before I'll leave it this way for
now until we figure out something better.

On Fri, Mar 3, 2017 at 3:46 PM Jim Ingham <jingham at apple.com> wrote:

> Yeah, looks like you need to keep DumpHexBytes in DumpDataExtractor.
> That's a little weird because it doesn't actually take a DataExtractor,
> but...
>
> Jim
>
> > On Mar 3, 2017, at 3:45 PM, Zachary Turner via lldb-commits <
> lldb-commits at lists.llvm.org> wrote:
> >
> > I can fix that, I didn't see it in my tests because it doesn't compile
> that file on Windows.  I'll get a fix in shortly.
> >
> > On Fri, Mar 3, 2017 at 3:38 PM Tim Hammerquist <thammerquist at apple.com>
> wrote:
> > Sure, Zachary. I try to step in when I can.
> >
> > Now that you mention it, your r296910 (DumpDataExtractor) patch is
> breaking building CommunicationKDP.cpp in MacOSX-Kernel. Namely, the
> removal of Dump/DumpHexBytes from DataExtractor. I've coordinated with the
> Apple LLDB team on how they want to follow up on that, but they might want
> to reach out to you about it.
> >
> > -Tim
> >
> >> On 3 Mar 2017, at 15:33, Zachary Turner <zturner at google.com> wrote:
> >>
> >> Thanks Tim!
> >>
> >> I have a few more file moves coming up as well, sorry for the trouble!
> >>
> >> On Fri, Mar 3, 2017 at 3:29 PM Tim Hammerquist via lldb-commits <
> lldb-commits at lists.llvm.org> wrote:
> >> Author: penryu
> >> Date: Fri Mar  3 17:17:29 2017
> >> New Revision: 296925
> >>
> >> URL: http://llvm.org/viewvc/llvm-project?rev=296925&view=rev
> >> Log:
> >> Fix Darwin failures introduced in r296909
> >>
> >> Modified:
> >>     lldb/trunk/lldb.xcodeproj/project.pbxproj
> >>
> >> Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj
> >> URL:
> http://llvm.org/viewvc/llvm-project/lldb/trunk/lldb.xcodeproj/project.pbxproj?rev=296925&r1=296924&r2=296925&view=diff
> >>
> ==============================================================================
> >> --- lldb/trunk/lldb.xcodeproj/project.pbxproj (original)
> >> +++ lldb/trunk/lldb.xcodeproj/project.pbxproj Fri Mar  3 17:17:29 2017
> >> @@ -2052,7 +2052,7 @@
> >>                 26BC7D6310F1B77400F91463 /* FileSpecList.h */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h;
> name = FileSpecList.h; path = include/lldb/Core/FileSpecList.h; sourceTree
> = "<group>"; };
> >>                 26BC7D6510F1B77400F91463 /* IOStreamMacros.h */ = {isa
> = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h;
> name = IOStreamMacros.h; path = include/lldb/Core/IOStreamMacros.h;
> sourceTree = "<group>"; };
> >>                 26BC7D6710F1B77400F91463 /* Listener.h */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h;
> name = Listener.h; path = include/lldb/Core/Listener.h; sourceTree =
> "<group>"; };
> >> -               26BC7D6810F1B77400F91463 /* Log.h */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h;
> name = Log.h; path = include/lldb/Core/Log.h; sourceTree = "<group>"; };
> >> +               26BC7D6810F1B77400F91463 /* Log.h */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h;
> name = Log.h; path = include/lldb/Utility/Log.h; sourceTree = "<group>"; };
> >>                 26BC7D6910F1B77400F91463 /* Mangled.h */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h;
> name = Mangled.h; path = include/lldb/Core/Mangled.h; sourceTree =
> "<group>"; };
> >>                 26BC7D6A10F1B77400F91463 /* Module.h */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h;
> name = Module.h; path = include/lldb/Core/Module.h; sourceTree = "<group>";
> };
> >>                 26BC7D6B10F1B77400F91463 /* ModuleChild.h */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h;
> name = ModuleChild.h; path = include/lldb/Core/ModuleChild.h; sourceTree =
> "<group>"; };
> >> @@ -2139,7 +2139,7 @@
> >>                 26BC7E7910F1B85900F91463 /* Event.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = Event.cpp; path = source/Core/Event.cpp; sourceTree = "<group>"; };
> >>                 26BC7E7B10F1B85900F91463 /* FileSpecList.cpp */ = {isa
> = PBXFileReference; fileEncoding = 4; lastKnownFileType =
> sourcecode.cpp.cpp; name = FileSpecList.cpp; path =
> source/Core/FileSpecList.cpp; sourceTree = "<group>"; };
> >>                 26BC7E7E10F1B85900F91463 /* Listener.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = Listener.cpp; path = source/Core/Listener.cpp; sourceTree =
> "<group>"; };
> >> -               26BC7E7F10F1B85900F91463 /* Log.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = Log.cpp; path = source/Core/Log.cpp; sourceTree = "<group>"; };
> >> +               26BC7E7F10F1B85900F91463 /* Log.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = Log.cpp; path = source/Utility/Log.cpp; sourceTree = "<group>"; };
> >>                 26BC7E8010F1B85900F91463 /* Mangled.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = Mangled.cpp; path = source/Core/Mangled.cpp; sourceTree = "<group>";
> };
> >>                 26BC7E8110F1B85900F91463 /* Module.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = Module.cpp; path = source/Core/Module.cpp; sourceTree = "<group>"; };
> >>                 26BC7E8210F1B85900F91463 /* ModuleChild.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = ModuleChild.cpp; path = source/Core/ModuleChild.cpp; sourceTree =
> "<group>"; };
> >> @@ -2335,8 +2335,8 @@
> >>                 33E5E8411A672A240024ED68 /* StringConvert.cpp */ = {isa
> = PBXFileReference; fileEncoding = 4; lastKnownFileType =
> sourcecode.cpp.cpp; path = StringConvert.cpp; sourceTree = "<group>"; };
> >>                 33E5E8451A6736D30024ED68 /* StringConvert.h */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h;
> name = StringConvert.h; path = include/lldb/Host/StringConvert.h;
> sourceTree = SOURCE_ROOT; };
> >>                 3F5E8AF31A40D4A500A73232 /* PipeBase.h */ = {isa =
> PBXFileReference; lastKnownFileType = sourcecode.c.h; name = PipeBase.h;
> path = include/lldb/Host/PipeBase.h; sourceTree = "<group>"; };
> >> -               3F8160A51AB9F7DD001DA9DF /* Logging.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = Logging.cpp; path = source/Core/Logging.cpp; sourceTree = "<group>";
> };
> >> -               3F8160A71AB9F809001DA9DF /* Logging.h */ = {isa =
> PBXFileReference; lastKnownFileType = sourcecode.c.h; name = Logging.h;
> path = include/lldb/Core/Logging.h; sourceTree = "<group>"; };
> >> +               3F8160A51AB9F7DD001DA9DF /* Logging.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = Logging.cpp; path = source/Utility/Logging.cpp; sourceTree =
> "<group>"; };
> >> +               3F8160A71AB9F809001DA9DF /* Logging.h */ = {isa =
> PBXFileReference; lastKnownFileType = sourcecode.c.h; name = Logging.h;
> path = include/lldb/Utility/Logging.h; sourceTree = "<group>"; };
> >>                 3F8169181ABA2419001DA9DF /* NameMatches.cpp */ = {isa =
> PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp;
> name = NameMatches.cpp; path = source/Utility/NameMatches.cpp; sourceTree =
> "<group>"; };
> >>                 3F81691C1ABA242B001DA9DF /* NameMatches.h */ = {isa =
> PBXFileReference; lastKnownFileType = sourcecode.c.h; name = NameMatches.h;
> path = include/lldb/Utility/NameMatches.h; sourceTree = "<group>"; };
> >>                 3F81692A1ABB7A16001DA9DF /* SystemInitializerFull.cpp
> */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType =
> sourcecode.cpp.cpp; name = SystemInitializerFull.cpp; path =
> source/API/SystemInitializerFull.cpp; sourceTree = "<group>"; };
> >>
> >>
> >> _______________________________________________
> >> lldb-commits mailing list
> >> lldb-commits at lists.llvm.org
> >> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
> >
> > _______________________________________________
> > lldb-commits mailing list
> > lldb-commits at lists.llvm.org
> > http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20170303/c4f1efea/attachment.html>


More information about the lldb-commits mailing list