Skip to content
  1. Mar 31, 2011
  2. Mar 30, 2011
    • Ted Kremenek's avatar
    • Bill Wendling's avatar
      * The DSE code that tested for overlapping needed to take into account the fact · 5034159c
      Bill Wendling authored
        that one of the numbers is signed while the other is unsigned. This could lead
        to a wrong result when the signed was promoted to an unsigned int.
      
      * Add the data layout line to the testcase so that it will test the appropriate
        thing.
      
      Patch by David Terei!
      
      llvm-svn: 128577
      5034159c
    • Johnny Chen's avatar
      A a simple test file for some lldb 'platform' commands. · c6401791
      Johnny Chen authored
      Add a missing result.SetStatus() stmt to the CommandObjectPlatformList::Execute() impl.
      
      llvm-svn: 128575
      c6401791
    • Akira Hatanaka's avatar
      fixed typo · 4e9ca1b3
      Akira Hatanaka authored
      llvm-svn: 128574
      4e9ca1b3
    • Ted Kremenek's avatar
      723cdf16
    • Johnny Chen's avatar
      Add a test directory. · 12bc549d
      Johnny Chen authored
      llvm-svn: 128572
      12bc549d
    • Caroline Tice's avatar
      · 55bff33b
      Caroline Tice authored
      Fill in code for EmulateSTRDImm and EmulateSTRDReg, to emulate the
      STRD (immediate) and STRD (register) instructions.
      
      llvm-svn: 128570
      55bff33b
    • Johnny Chen's avatar
      Add an option to specify the symbols to disassemble instead of the existing... · 318e7ba6
      Johnny Chen authored
      Add an option to specify the symbols to disassemble instead of the existing 'num of symbols to disassemble'
      option.  If both are present, the 'symbols to disassemble' overrides the 'num of symbols to disassemble'.
      
      An example usage:
      
      $ ./lldb-disasm.py -C 'platform create remote-ios' -e /Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS4.3.sdk/usr/lib/libSystem.B.dylib -o '-r -n' -s vprintf -s acosf_special
      
      llvm-svn: 128569
      318e7ba6
    • Jakob Stoklund Olesen's avatar
      Don't add the same analysis implementation pair twice. · aab97dce
      Jakob Stoklund Olesen authored
      Prevent infinite growth of the list.
      
      Patch by José Fonseca!
      
      llvm-svn: 128568
      aab97dce
    • Jakob Stoklund Olesen's avatar
      Reset StringMap's NumTombstones on clears and rehashes. · 846f9508
      Jakob Stoklund Olesen authored
      StringMap was not properly updating NumTombstones after a clear or rehash.
      
      This was not fatal until now because the table was growing faster than
      NumTombstones could, but with the previous change of preventing infinite
      growth of the table the invariant (NumItems + NumTombstones <= NumBuckets)
      stopped being observed, causing infinite loops in certain situations.
      
      Patch by José Fonseca!
      
      llvm-svn: 128567
      846f9508
    • Jakob Stoklund Olesen's avatar
      Prevent infinite growth of SmallPtrSet instances. · bdc1b012
      Jakob Stoklund Olesen authored
      Rehash but don't grow when full of tombstones.
      
      Patch by José Fonseca!
      
      llvm-svn: 128566
      bdc1b012
    • Jakob Stoklund Olesen's avatar
      Prevent infinite growth of SmallMap instances. · f587f441
      Jakob Stoklund Olesen authored
      Rehash but don't grow when full of tombstones.
      
      Patch by José Fonseca!
      
      llvm-svn: 128565
      f587f441
    • Jakob Stoklund Olesen's avatar
      Prevent infinite growth of the DenseMap. · 5ca05e18
      Jakob Stoklund Olesen authored
      When the hash function uses object pointers all free entries eventually
      become tombstones as they are used at least once, regardless of the size.
      
      DenseMap cannot function with zero empty keys, so it double size to get
      get ridof the tombstones.
      
      However DenseMap never shrinks automatically unless it is cleared, so
      the net result is that certain tables grow infinitely.
      
      The solution is to make a fresh copy of the table without tombstones
      instead of doubling size, by simply calling grow with the current size.
      
      Patch by José Fonseca!
      
      llvm-svn: 128564
      5ca05e18
    • Greg Clayton's avatar
      Many improvements to the Platform base class and subclasses. The base Platform · 32e0a750
      Greg Clayton authored
      class now implements the Host functionality for a lot of things that make 
      sense by default so that subclasses can check:
      
      int
      PlatformSubclass::Foo ()
      {
          if (IsHost())
              return Platform::Foo (); // Let the platform base class do the host specific stuff
          
          // Platform subclass specific code...
          int result = ...
          return result;
      }
      
      Added new functions to the platform:
      
          virtual const char *Platform::GetUserName (uint32_t uid);
          virtual const char *Platform::GetGroupName (uint32_t gid);
      
      The user and group names are cached locally so that remote platforms can avoid
      sending packets multiple times to resolve this information.
      
      Added the parent process ID to the ProcessInfo class. 
      
      Added a new ProcessInfoMatch class which helps us to match processes up
      and changed the Host layer over to using this new class. The new class allows
      us to search for processs:
      1 - by name (equal to, starts with, ends with, contains, and regex)
      2 - by pid
      3 - And further check for parent pid == value, uid == value, gid == value, 
          euid == value, egid == value, arch == value, parent == value.
          
      This is all hookup up to the "platform process list" command which required
      adding dumping routines to dump process information. If the Host class 
      implements the process lookup routines, you can now lists processes on 
      your local machine:
      
      machine1.foo.com % lldb
      (lldb) platform process list 
      PID    PARENT USER       GROUP      EFF USER   EFF GROUP  TRIPLE                   NAME
      ====== ====== ========== ========== ========== ========== ======================== ============================
      99538  1      username   usergroup  username   usergroup  x86_64-apple-darwin      FileMerge
      94943  1      username   usergroup  username   usergroup  x86_64-apple-darwin      mdworker
      94852  244    username   usergroup  username   usergroup  x86_64-apple-darwin      Safari
      94727  244    username   usergroup  username   usergroup  x86_64-apple-darwin      Xcode
      92742  92710  username   usergroup  username   usergroup  i386-apple-darwin        debugserver
      
      
      This of course also works remotely with the lldb-platform:
      
      machine1.foo.com % lldb-platform --listen 1234
      
      machine2.foo.com % lldb
      (lldb) platform create remote-macosx
        Platform: remote-macosx
       Connected: no
      (lldb) platform connect connect://localhost:1444
        Platform: remote-macosx
          Triple: x86_64-apple-darwin
      OS Version: 10.6.7 (10J869)
          Kernel: Darwin Kernel Version 10.7.0: Sat Jan 29 15:17:16 PST 2011; root:xnu-1504.9.37~1/RELEASE_I386
        Hostname: machine1.foo.com
       Connected: yes
      (lldb) platform process list 
      PID    PARENT USER       GROUP      EFF USER   EFF GROUP  TRIPLE                   NAME
      ====== ====== ========== ========== ========== ========== ======================== ============================
      99556  244    username   usergroup  username   usergroup  x86_64-apple-darwin      trustevaluation
      99548  65539  username   usergroup  username   usergroup  x86_64-apple-darwin      lldb
      99538  1      username   usergroup  username   usergroup  x86_64-apple-darwin      FileMerge
      94943  1      username   usergroup  username   usergroup  x86_64-apple-darwin      mdworker
      94852  244    username   usergroup  username   usergroup  x86_64-apple-darwin      Safari
      
      The lldb-platform implements everything with the Host:: layer, so this should
      "just work" for linux. I will probably be adding more stuff to the Host layer
      for launching processes and attaching to processes so that this support should
      eventually just work as well.
      
      Modified the target to be able to be created with an architecture that differs
      from the main executable. This is needed for iOS debugging since we can have
      an "armv6" binary which can run on an "armv7" machine, so we want to be able
      to do:
      
      % lldb
      (lldb) platform create remote-ios
      (lldb) file --arch armv7 a.out
      
      Where "a.out" is an armv6 executable. The platform then can correctly decide
      to open all "armv7" images for all dependent shared libraries.
      
      Modified the disassembly to show the current PC value. Example output:
      
      (lldb) disassemble --frame
      a.out`main:
         0x1eb7:  pushl  %ebp
         0x1eb8:  movl   %esp, %ebp
         0x1eba:  pushl  %ebx
         0x1ebb:  subl   $20, %esp
         0x1ebe:  calll  0x1ec3                   ; main + 12 at test.c:18
         0x1ec3:  popl   %ebx
      -> 0x1ec4:  calll  0x1f12                   ; getpid
         0x1ec9:  movl   %eax, 4(%esp)
         0x1ecd:  leal   199(%ebx), %eax
         0x1ed3:  movl   %eax, (%esp)
         0x1ed6:  calll  0x1f18                   ; printf
         0x1edb:  leal   213(%ebx), %eax
         0x1ee1:  movl   %eax, (%esp)
         0x1ee4:  calll  0x1f1e                   ; puts
         0x1ee9:  calll  0x1f0c                   ; getchar
         0x1eee:  movl   $20, (%esp)
         0x1ef5:  calll  0x1e6a                   ; sleep_loop at test.c:6
         0x1efa:  movl   $12, %eax
         0x1eff:  addl   $20, %esp
         0x1f02:  popl   %ebx
         0x1f03:  leave
         0x1f04:  ret
         
      This can be handy when dealing with the new --line options that was recently
      added:
      
      (lldb) disassemble --line
      a.out`main + 13 at test.c:19
         18  	{
      -> 19  		printf("Process: %i\n\n", getpid());
         20  	    puts("Press any key to continue..."); getchar();
      -> 0x1ec4:  calll  0x1f12                   ; getpid
         0x1ec9:  movl   %eax, 4(%esp)
         0x1ecd:  leal   199(%ebx), %eax
         0x1ed3:  movl   %eax, (%esp)
         0x1ed6:  calll  0x1f18                   ; printf
      
      Modified the ModuleList to have a lookup based solely on a UUID. Since the
      UUID is typically the MD5 checksum of a binary image, there is no need
      to give the path and architecture when searching for a pre-existing
      image in an image list.
      
      Now that we support remote debugging a bit better, our lldb_private::Module
      needs to be able to track what the original path for file was as the platform
      knows it, as well as where the file is locally. The module has the two 
      following functions to retrieve both paths:
      
      const FileSpec &Module::GetFileSpec () const;
      const FileSpec &Module::GetPlatformFileSpec () const;
      
      llvm-svn: 128563
      32e0a750
    • Jakob Stoklund Olesen's avatar
      Fix evil VirtRegRewriter bug. · ae917a37
      Jakob Stoklund Olesen authored
      The rewriter can keep track of multiple stack slots in the same register if they
      happen to have the same value. When an instruction modifies a stack slot by
      defining a register that is mapped to a stack slot, other stack slots in that
      register are no longer valid.
      
      This is a very rare problem, and I don't have a simple test case. I get the
      impression that VirtRegRewriter knows it is about to be deleted, inventing a
      last opaque problem.
      
      <rdar://problem/9204040>
      
      llvm-svn: 128562
      ae917a37
    • Jakob Stoklund Olesen's avatar
    • Argyrios Kyrtzidis's avatar
    • Caroline Tice's avatar
      · 23443245
      Caroline Tice authored
      Fill in EmulateLDRDRegister to emulate LDRD (register) instruction.
      
      Remove stubs for functions not-to-be-implemented at the moment.
      
      llvm-svn: 128559
      23443245
    • Johnny Chen's avatar
      Minor import statement change. · 90bb9051
      Johnny Chen authored
      llvm-svn: 128558
      90bb9051
Loading