Skip to content
  1. Nov 14, 2013
  2. Nov 01, 2013
  3. Oct 31, 2013
  4. Oct 29, 2013
  5. Oct 19, 2013
  6. Oct 17, 2013
    • Richard Mitton's avatar
      Added support for reading thread-local storage variables, as defined using the __thread modifier. · 0a558357
      Richard Mitton authored
      To make this work this patch extends LLDB to:
      
      - Explicitly track the link_map address for each module. This is effectively the module handle, not sure why it wasn't already being stored off anywhere. As an extension later, it would be nice if someone were to add support for printing this as part of the modules list.
      
      - Allow reading the per-thread data pointer via ptrace. I have added support for Linux here. I'll be happy to add support for FreeBSD once this is reviewed. OS X does not appear to have __thread variables, so maybe we don't need it there. Windows support should eventually be workable along the same lines.
      
      - Make DWARF expressions track which module they originated from.
      
      - Add support for the DW_OP_GNU_push_tls_address DWARF opcode, as generated by gcc and recent versions of clang. Earlier versions of clang (such as 3.2, which is default on Ubuntu right now) do not generate TLS debug info correctly so can not be supported here.
      
      - Understand the format of the pthread DTV block. This is where it gets tricky. We have three basic options here:
      
        1) Call "dlinfo" or "__tls_get_addr" on the inferior and ask it directly. However this won't work on core dumps, and generally speaking it's not a good idea for the debugger to call functions itself, as it has the potential to not work depending on the state of the target.
      
        2) Use libthread_db. This is what GDB does. However this option requires having a version of libthread_db on the host cross-compiled for each potential target. This places a large burden on the user, and would make it very hard to cross-debug from Windows to Linux, for example. Trying to build a library intended exclusively for one OS on a different one is not pleasant. GDB sidesteps the problem and asks the user to figure it out.
      
        3) Parse the DTV structure ourselves. On initial inspection this seems to be a bad option, as the DTV structure (the format used by the runtime to manage TLS data) is not in fact a kernel data structure, it is implemented entirely in useerland in libc. Therefore the layout of it's fields are version and OS dependent, and are not standardized.
      
        However, it turns out not to be such a problem. All OSes use basically the same algorithm (a per-module lookup table) as detailed in Ulrich Drepper's TLS ELF ABI document, so we can easily write code to decode it ourselves. The only question therefore is the exact field layouts required. Happily, the implementors of libpthread expose the structure of the DTV via metadata exported as symbols from the .so itself, designed exactly for this kind of thing. So this patch simply reads that metadata in, and re-implements libthread_db's algorithm itself. We thereby get cross-platform TLS lookup without either requiring third-party libraries, while still being independent of the version of libpthread being used.
      
      Test case included.
      
      llvm-svn: 192922
      0a558357
  7. Oct 05, 2013
    • Enrico Granata's avatar
      <rdar://problem/12042982> · a29cb0ba
      Enrico Granata authored
      This radar extends the notion of one-liner summaries to automagically apply in a few interesting cases
      
      More specifically, this checkin changes the printout of ValueObjects to print on one-line (as if type summary add -c had been applied) iff:
      this ValueObject does not have a summary
      its children have no synthetic children
      its children are not a non-empty base class without a summary
      its children do not have a summary that asks for children to show up
      the aggregate length of all the names of all the children is <= 50 characters
      you did not ask to see the types during a printout
      your pointer depth is 0
      
      This is meant to simplify the way LLDB shows data on screen for small structs and similarly compact data types (e.g. std::pair<int,int> anyone?)
      
      Feedback is especially welcome on how the feature feels and corner cases where we should apply this printout and don't (or viceversa, we are applying it when we shouldn't be)
      
      llvm-svn: 191996
      a29cb0ba
  8. Sep 13, 2013
    • Ed Maste's avatar
      test: Update FreeBSD failure decorators · 34bdbbdf
      Ed Maste authored
      llvm.org/pr15261 missing size for static arrays
      llvm.org/pr15278 expressions generating signals
      llvm.org/pr15824 thread states aren't properly maintained
      llvm.org/pr16696 threaded inferior debugging not yet on FreeBSD
      llvm.org/pr17214 inline stepping fails on FreeBSD
      llvm.org/pr17225 Clang assertion failure
      llvm.org/pr17226 frame info lost after failed expression evaluation
      llvm.org/pr17228 test timeout
      
      The first three are existing Linux issues that also affect FreeBSD.
      
      llvm-svn: 190698
      34bdbbdf
    • Ed Maste's avatar
      Rename test class to match test target language · a837c822
      Ed Maste authored
      llvm-svn: 190695
      a837c822
  9. Sep 09, 2013
  10. Aug 30, 2013
  11. Aug 27, 2013
    • Daniel Malea's avatar
      merge lldb-platform-work branch (and assorted fixes) into trunk · e0f8f574
      Daniel Malea authored
          Summary:
          This merge brings in the improved 'platform' command that knows how to
          interface with remote machines; that is, query OS/kernel information, push
          and pull files, run shell commands, etc... and implementation for the new
          communication packets that back that interface, at least on Darwin based
          operating systems via the POSIXPlatform class. Linux support is coming soon.
      
          Verified the test suite runs cleanly on Linux (x86_64), build OK on Mac OS
          X Mountain Lion.
      
          Additional improvements (not in the source SVN branch 'lldb-platform-work'):
          - cmake build scripts for lldb-platform
          - cleanup test suite
          - documentation stub for qPlatform_RunCommand
          - use log class instead of printf() directly
          - reverted work-in-progress-looking changes from test/types/TestAbstract.py that work towards running the test suite remotely.
          - add new logging category 'platform'
      
          Reviewers: Matt Kopec, Greg Clayton
      
          Review: http://llvm-reviews.chandlerc.com/D1493
      
      llvm-svn: 189295
      e0f8f574
  12. Aug 05, 2013
  13. Jul 31, 2013
  14. Jul 30, 2013
  15. Jul 24, 2013
  16. Jul 02, 2013
  17. Jun 17, 2013
  18. Jun 12, 2013
  19. Jun 06, 2013
  20. May 31, 2013
  21. May 15, 2013
  22. May 09, 2013
  23. May 03, 2013
  24. Apr 27, 2013
  25. Apr 24, 2013
  26. Apr 19, 2013
    • Ashok Thirumurthi's avatar
      Improved tests for nested structs when anonymous structs are involved. · f6cd60ae
      Ashok Thirumurthi authored
      - New tests can fail on OS/X and Linux, and illustrate that the compiler
      used to generate the DWARF can result in lldb providing clang with an
      external AST source that doesn't describe all required struct fields.
      
      - Also includes test coverage for expressions with structs that do work on Linux- Also includes a test for dereferencing a null pointer to a struct, which works on OS/X rather than complaining, and complains on Linux with an upstream error that is really a side issue.
      
      Thanks to Samuel, Andy and Daniel for their input.
      
      llvm-svn: 179884
      f6cd60ae
  27. Mar 28, 2013
  28. Mar 15, 2013
    • Matt Kopec's avatar
      Add icc support to the test suite · 0de53f06
      Matt Kopec authored
      -adds icc to the lit of compilers to run the tests
      -adds icc test decorators
      -skip TestAnonymous.py for icc
      
      Patch by Ashok Thirumurthi.
      
      llvm-svn: 177174
      0de53f06
  29. Mar 06, 2013
    • Andrew Kaylor's avatar
      Skipping TestConstVariables on Linux. · 7bdeab59
      Andrew Kaylor authored
      The test was marked as expectedFailureLinux, but now it passes with gcc and some versions of clang.  Newer versions of clang introduce a failure due to bad DWARF information.
      
      llvm-svn: 176581
      7bdeab59
    • Greg Clayton's avatar
      Fixed enum printing for negative enums. There previously was no testing to... · 5d14fc06
      Greg Clayton authored
      Fixed enum printing for negative enums. There previously was no testing to validate that enum values were being displayed correctly.
      
      Also added C++11 enum test cases to cover enums as int8_t, int16_t int32_t, int64_t, uint8_t, uint16_t, uint32_t, and uint64_t both for DWARF and dSYM cases. The DWARF being emitted by clang is missing the enum integer type, but the code is now ready to accept and deal with the integral type if it is supplied.
      
      llvm-svn: 176548
      5d14fc06
  30. Mar 05, 2013
  31. Feb 28, 2013
  32. Feb 27, 2013
  33. Feb 26, 2013
    • Jim Ingham's avatar
      Fix the .categories, it had "dataformatter" not "dataformatters". · d882998e
      Jim Ingham authored
      Remove the getCategory from TestDataFormatterObjC.py, since it was superceded by the .categories file, 
      and didn't work anyway (getCategories currently has to be a method on the test class, not on the test.)
      Add a "basic_process" category, and start to find some tests for simple process running sniff tests.
      
      llvm-svn: 176061
      d882998e
  34. Feb 22, 2013
  35. Feb 20, 2013
    • Daniel Malea's avatar
      Fix invalid TestBitfields case (thanks Filipe!), and xfail one case due to GCC compiler bug. · c6a70063
      Daniel Malea authored
      Turns out unnamed bit fields should not be accessed ever in C (C99 section 4.7.8) or C++
      (C++11 section 9.6). 
      
      Unrelated to the above, this commit marks TestBitfields as expected-to-fail with GCC as 4.7
      has problems with padding... the size of the struct with a uint32_t member is reported as 12
      when GCC is used. Clang emits correct code.
      
      Parts of this patch prepared by Filipe Cabecinhas!
      
      llvm-svn: 175666
      c6a70063
Loading