1. Jul 08, 2016
  2. Jul 07, 2016
    • Etienne Bergeron's avatar
      [compiler-rt] Fix AllocatorCacheDeallocNewThread unittest · ec4b0e4e
      Etienne Bergeron authored
      Summary:
      The unittest was not freeing the mapped memory.
      
      
      ```
      Repeating all tests (iteration 1) . . .
      
      Note: Google Test filter = Allocator.AllocatorCacheDeallocNewThread
      [==========] Running 1 test from 1 test case.
      [----------] Global test environment set-up.
      [----------] 1 test from Allocator
      [ RUN      ] Allocator.AllocatorCacheDeallocNewThread
      [       OK ] Allocator.AllocatorCacheDeallocNewThread (3 ms)
      [----------] 1 test from Allocator (4 ms total)
      
      [----------] Global test environment tear-down
      [==========] 1 test from 1 test case ran. (5 ms total)
      [  PASSED  ] 1 test.
      
      Repeating all tests (iteration 2) . . .
      
      Note: Google Test filter = Allocator.AllocatorCacheDeallocNewThread
      [==========] Running 1 test from 1 test case.
      [----------] Global test environment set-up.
      [----------] 1 test from Allocator
      [ RUN      ] Allocator.AllocatorCacheDeallocNewThread
      ==4504==WARNING: SanitizerTool failed to mprotect 0x010000003000 (1099511640064) bytes at 0x010000000000 (error code: 48
      7)
      ==4504==Sanitizer CHECK failed: D:/src/llvm/llvm/projects/compiler-rt/lib\sanitizer_common/sanitizer_allocator.h:329 ((kSpaceBeg)) == ((reinterpret_cast<uptr>( MmapFixedNoAccess(kSpaceBeg, TotalSpaceSize)))) (1099511627776, 0)
      ```
      
      Reviewers: rnk
      
      Subscribers: llvm-commits, kubabrecka, chrisha
      
      Differential Revision: http://reviews.llvm.org/D22094
      
      llvm-svn: 274764
      ec4b0e4e
    • Pavel Labath's avatar
      [LLGS] Work around an adb bug on Android <=M · ca92aed5
      Pavel Labath authored
      On android M it can happen that we get a ETXTBSY, when we try to launch the inferior. Sleeping
      and retrying should help us get more stable results.
      
      llvm-svn: 274763
      ca92aed5
    • Pavel Labath's avatar
      Bump up timeout in TestCallWithTimeout · 613cce83
      Pavel Labath authored
      remote targets need a bit more time to get their act together
      
      llvm-svn: 274762
      613cce83
    • Zhan Jun Liau's avatar
      [SystemZ] Fix regression when handling conditional calls · a5d60afc
      Zhan Jun Liau authored
      Summary:
      A regression showed up in node.js when handling conditional calls.
      Fix the regression by recognizing external symbols as a possible
      operand type in CallJG.
      
      Reviewers: koriakin
      
      Subscribers: llvm-commits
      
      Differential Revision: http://reviews.llvm.org/D22054
      
      llvm-svn: 274761
      a5d60afc
    • Sanjay Patel's avatar
      save type in local var; NFCI · 25600f39
      Sanjay Patel authored
      llvm-svn: 274760
      25600f39
    • Benjamin Kramer's avatar
      [clang-rename] add basic vim integration · 3a12ca57
      Benjamin Kramer authored
      This patch introduces basic Vim integration for clang-rename tool.
      
      For setup reference see clang-rename/tool/clang-rename.py
      
      Patch by Kirill Bobyrev!
      
      Differential revision: http://reviews.llvm.org/D22087
      
      llvm-svn: 274759
      3a12ca57
    • Sjoerd Meijer's avatar
      Addressing post-commit comments for not rewriting fputs: · 7435a910
      Sjoerd Meijer authored
      moved the optimise for size check inside function optimizeFPuts.
      
      llvm-svn: 274758
      7435a910
    • Rafael Espindola's avatar
      Fix formating. NFC. · 64c32d6f
      Rafael Espindola authored
      llvm-svn: 274757
      64c32d6f
    • Valery Pykhtin's avatar
      [AMDGPU] fix ds_write_src2 encoding (bz26027) · af8b1bdd
      Valery Pykhtin authored
      Differential revision: http://reviews.llvm.org/D22041
      
      llvm-svn: 274756
      af8b1bdd
    • Rafael Espindola's avatar
      Use "using namespace llvm" like the rest of ELF lld. · d10f0b1e
      Rafael Espindola authored
      llvm-svn: 274755
      d10f0b1e
    • Rafael Espindola's avatar
      Don't crash trying to relax 32 loads on COFF. · b34cba97
      Rafael Espindola authored
      Fixes pr28452.
      
      llvm-svn: 274754
      b34cba97
    • Sjoerd Meijer's avatar
      Code size optimisation: don't rewrite fputs to fwrite when optimising for size · 17c08dc7
      Sjoerd Meijer authored
      because fwrite requires more arguments and thus extra MOVs are required.
      
      llvm-svn: 274753
      17c08dc7
    • Rafael Espindola's avatar
      This now works on unix too. · 4225342e
      Rafael Espindola authored
      With this there are no unsupported tests on linux in test/ELF :-)
      
      llvm-svn: 274752
      4225342e
    • Nico Weber's avatar
      [clang-tidy] Add dependency on clang-headers · 1e348b33
      Nico Weber authored
      Currently, to be able to process a source file including e.g. stddef.h with
      clang-tidy, one has to build both clang-tidy and the clang-headers target.
      Since stddef.h is needed for virtually any source file, let clang-tidy depend
      on clang-headers, so that it Just Works after it has been built.
      
      http://reviews.llvm.org/D22046
      
      llvm-svn: 274751
      1e348b33
    • Ravitheja Addepally's avatar
      Fix for PrintStackTraces · b3b972a5
      Ravitheja Addepally authored
      Summary:
      The issue arises due to the wrong unwinder used for the first
      stack frame, where the default unwinder returns erroneous frame
      whereas the fallback would have given the correct frame had it
      been used.
      The following fix consists of two parts ->
       1) The first part changes the unwinding strategy, earlier the
          default unwinder was used to get 2 more stack frames and if it
          failed a fallback unwinder was used. Now we try to obtain as many
          frames (max 10) as we can get from default unwinder and also
          fallback unwinder and use the one that gives more number of frames.
          
       2) Normally unwindplans are assosciated with functions and the
          row to be used is obtained from the offset (obtained from the low_pc
          of the function symbol). Sometimes it may occur that the unwindplan
          is assosciated to the complete Elf section in which case the offset
          calculation would be wrong as the debugger uses the same offset originally
          obtained from the function symbol. Hence this offset is recalculated.
      
      Reviewers: tberghammer, lldb-commits, labath, jasonmolenda
      
      Subscribers: jingham
      
      Differential Revision: http://reviews.llvm.org/D21221
      
      llvm-svn: 274750
      b3b972a5
    • Kuba Brecka's avatar
      [tsan] Avoid false positives with GCD data callbacks · 4446c216
      Kuba Brecka authored
      This patch adds synchronization between the creation of the GCD data object and destructor’s execution. It’s far from perfect, because ideally we’d want to synchronize the destruction of the last reference (via dispatch_release) and the destructor’s execution, but intercepting objc_release is problematic.
      
      Differential Revision: http://reviews.llvm.org/D21990
      
      llvm-svn: 274749
      4446c216
    • Simon Pilgrim's avatar
      Fix "not all control paths return a value" warning on MSVC · 474c6421
      Simon Pilgrim authored
      This time without causing a 'all enums handled' warning on other compilers.
      
      llvm-svn: 274748
      474c6421
    • Simon Pilgrim's avatar
      Fix "not all control paths return a value" warning on MSVC · c6db895e
      Simon Pilgrim authored
      llvm-svn: 274746
      c6db895e
    • Alexey Bataev's avatar
      [OPENMP] Do not create helper expressions in dependent contexts, NFC. · be8b8b5b
      Alexey Bataev authored
      OpenMP relies on some helper expressions generated during semantic
      analysis. But they are required only for codegen and not required in
      dependent contexts. Patch removes generation of some of helper
      expressions.
      
      llvm-svn: 274745
      be8b8b5b
    • Asiri Rathnayake's avatar
      [libunwind][ARM] Improve unwinder stack usage - Make WMMX support optional · f33c3423
      Asiri Rathnayake authored
      These registers are only available on a limited set of ARM targets (those
      based on XScale). Other targets should not have to pay the cost of these.
      
      This patch shaves off about ~300 bytes of stack usage and ~1KB of code-size.
      
      Differential revision: http://reviews.llvm.org/D21991
      Reviewers: bcraig, compnerd
      
      Change-Id: I2d7a1911a193bd70b123e78747e1a7d1482463c7
      llvm-svn: 274744
      f33c3423
    • Tamas Berghammer's avatar
      Try to fix the OSX build with old SDK after r274725 · 130802b3
      Tamas Berghammer authored
      llvm-svn: 274743
      130802b3
    • Diana Picus's avatar
      [ARM] Do not test for CPUs, use SubtargetFeatures. Also remove 1 flag · 575f2bb2
      Diana Picus authored
      This is a follow-up for r273544.
      
      The end goal is to get rid of the isSwift / isCortexXY / isWhatever methods.
      
      This commit also removes a command line flag that isn't used in any of the tests:
      check-vmlx-hazards. It can be replaced easily with the mattr mechanism, since
      this is now a subtarget feature.
      
      There is still some work left regarding FeatureExpandMLx. In the past MLx
      expansion was enabled for subtargets with hasVFP2(), until r129775 [1] switched
      from that to isCortexA9, without too much justification.
      
      In spite of that, the code performing MLx expansion still contains calls to
      isSwift/isLikeA9, although the results of those are pretty clear given that
      we're only enabling it for the A9.
      
      We should try to enable it for all targets that have FeatureHasVMLxHazards, as
      it seems to be closely related to that behaviour, and if that is possible try to
      clean up the MLx expansion pass from all calls to isWhatever. This will require
      some performance testing, so it will be done in another patch.
      
      [1] http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20110418/119725.html
      
      Differential Revision: http://reviews.llvm.org/D21798
      
      llvm-svn: 274742
      575f2bb2
    • Howard Hellyer's avatar
      Implement GetMemoryRegions() for Linux and Mac OSX core files. · ad007563
      Howard Hellyer authored
      Summary:
      This patch fills in the implementation of GetMemoryRegions() on the Linux and Mac OS core file implementations of lldb_private::Process (ProcessElfCore::GetMemoryRegions and ProcessMachCore::GetMemoryRegions.) The GetMemoryRegions API was added under: http://reviews.llvm.org/D20565
      
      The patch re-uses the m_core_range_infos list that was recently added to implement GetMemoryRegionInfo in both ProcessElfCore and ProcessMachCore to ensure the returned regions match the regions returned by Process::GetMemoryRegionInfo(addr_t load_addr, MemoryRegionInfo &region_info).
      
      Reviewers: clayborg
      
      Subscribers: labath, lldb-commits
      
      Differential Revision: http://reviews.llvm.org/D21751
      
      llvm-svn: 274741
      ad007563
    • Chandler Carruth's avatar
      [LCG] Hoist the definitions of the stream operator friends to be inline · 168800c9
      Chandler Carruth authored
      friend definitions.
      
      Based on the experiments Sean Silva and Reid did, this seems the safest
      course of action and also will work around a questionable warning
      provided by GCC6 on the old form of the code. Thanks for Davide pointing
      out the issue and other suggesting ways to fix.
      
      llvm-svn: 274740
      168800c9