Started 2 yr 2 mo ago
Took 1 hr 42 min on green-dragon-15

Success Build #383 (Aug 14, 2017 1:38:25 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 310816
  • http://llvm.org/svn/llvm-project/cfe/trunk : 310817
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 310800
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 303903
  • http://llvm.org/svn/llvm-project/zorg/trunk : 310746
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 310761
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 310818
Changes
  1. [clangd] Fixed a data race.

    Summary:
    Calling addDocument after removeDocument could have resulted in an
    invalid program state (AST and Preamble for the valid document could
    have been incorrectly removed).
    This commit also includes an improved CppFile::cancelRebuild
    implementation that allows to cancel reparse without waiting for
    ongoing rebuild to finish.

    Reviewers: krasimir, bkramer, klimek

    Reviewed By: bkramer

    Subscribers: cfe-commits

    Differential Revision: https://reviews.llvm.org/D36397 (detail)
    by ibiryukov
  2. Fix memory leak in ToolChain::TranslateOpenMPTargetArgs

    rL310433 introduced a code path where DAL is not returned and must be freed.
    This change allows to run openmp-offload.c when Clang is built with ASan. (detail)
    by hahnfeld
  3. [ValueTracking] Revert r310583 which enabled functionality that still is
    causing compile time issues.

    Moreover, the patch *deleted* the flag in addition to changing the
    default, and links to a code review that doesn't even discuss the flag
    and just has an update to a Clang test case.

    I've followed up on the commit thread to ask for numbers on compile time
    at this point, leaving the flag in place until things stabilize, and
    pointing at specific code that seems to exhibit excessive compile time
    with this patch.

    Original commit message for r310583:
    """
    [ValueTracking] Enabling ValueTracking patch by default (recommit). Part 2.

    The original patch was an improvement to IR ValueTracking on
    non-negative integers. It has been checked in to trunk (D18777,
    r284022). But was disabled by default due to performance regressions.
    Perf impact has improved. The patch would be enabled by default.
    """" (detail)
    by chandlerc

Started by timer (2 times)

This run spent:

  • 1 hr 27 min waiting;
  • 1 hr 42 min build duration;
  • 3 hr 9 min total from scheduled to completion.
Test Result (no failures)