Started 6 hr 26 min ago
Took 4 hr 52 min on green-dragon-02

Failed Build #14895 (Oct 16, 2019 4:04:33 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 374992
  • http://llvm.org/svn/llvm-project/cfe/trunk : 374991
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 374977
  • http://llvm.org/svn/llvm-project/zorg/trunk : 374976
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 374854
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 374982
Changes
  1. Revert "[HardwareLoops] Optimisation remarks"

    while I investigate the PPC build bot failures.

    This reverts commit ad763751565b9663bc338fa2ca5ade86c6ca22ec. (detail)
    by sjoerdmeijer
  2. RewriteModernObjC - silence static analyzer getAs<> null dereference warnings. NFCI.

    The static analyzer is warning about potential null dereferences, but in these cases we should be able to use castAs<> directly and if not assert will fire for us. (detail)
    by rksimon
  3. [ARM] Add a register class for GPR pairs without SP and use it. NFCI

    Summary:
    Currently Thumb2InstrInfo.cpp uses a register class which is
    auto-generated by tablegen. Such approach is fragile because
    auto-generated classes might change when other register classes are
    added. For example, before https://reviews.llvm.org/D62667
    we were using GPRPair_with_gsub_1_in_rGPRRegClass, but had to
    change it to GPRPair_with_gsub_1_in_GPRwithAPSRnospRegClass
    because the former class stopped being generated (this did not change
    the functionality though).

    This patch adds a register class consisting of even-odd GPR register
    pairs from (R0, R1) to (R10, R11), which excludes (R12, SP) and uses
    it in Thumb2InstrInfo.cpp instead of
    GPRPair_with_gsub_1_in_GPRwithAPSRnospRegClass.

    Reviewers: ostannard, simon_tatham, dmgreen, efriedma

    Reviewed By: simon_tatham

    Subscribers: kristof.beyls, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69026 (detail)
    by miyuki
  4. CGDebugInfo - silence static analyzer dyn_cast<> null dereference warnings. NFCI.

    The static analyzer is warning about potential null dereferences, but in these cases we should be able to use cast<> directly and if not assert will fire for us. (detail)
    by rksimon
  5. CGExprConstant - silence static analyzer getAs<> null dereference warning. NFCI.

    The static analyzer is warning about a potential null dereference, but in these cases we should be able to use castAs<> directly and if not assert will fire for us. (detail)
    by rksimon
  6. CGBuiltin - silence static analyzer getAs<> null dereference warnings. NFCI.

    The static analyzer is warning about potential null dereferences, but in these cases we should be able to use castAs<> directly and if not assert will fire for us. (detail)
    by rksimon
  7. SimpleLoopUnswitch - fix uninitialized variable and null dereference warnings. NFCI. (detail)
    by rksimon
  8. Revert 374967 "[Concepts] ConceptSpecializationExprs mangling"

    This reverts commit 5e34ad109ced8dbdea9500ee28180315b2aeba3d.

    The mangling test fails on Windows:
    http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/15944

    It also fails on ppc64le:
    http://lab.llvm.org:8011/builders/clang-ppc64le-linux-lnt/builds/21092

    Also revert follow-up  374971 "Fix failing mangle-concept.cpp test."
    (it did not help on Win/ppc64le). (detail)
    by nico
  9. [AMDGPU] Extend the SI Load/Store optimizer

    Summary:
    Extend the SI Load/Store optimizer to merge MIMG load instructions. Handle
    different flavours of image_load and image_sample instructions.

    When the instructions of the same subclass differ only in dmask, merge
    them and update dmask accordingly.

    Reviewers: nhaehnle

    Reviewed By: nhaehnle

    Subscribers: arsenm, kzhuravl, jvesely, wdng, nhaehnle, yaxunl, dstuttard, tpr, t-tye, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64911 (detail)
    by Piotr Sobczak
  10. gn build: Merge r374982 (detail)
    by gnsyncbot
  11. [clangd] Add RemoveUsingNamespace tweak.

    Summary:
    Removes the 'using namespace' under the cursor and qualifies all accesses in the current file.
    E.g.:
      using namespace std;
      vector<int> foo(std::map<int, int>);
    Would become:
      std::vector<int> foo(std::map<int, int>);

    Subscribers: mgorny, ilya-biryukov, MaskRay, jkorous, mgrang, arphaman, kadircet, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D68562 (detail)
    by usaxena95
  12. [ARM][ParallelDSP] Change smlad insertion order

    Instead of inserting everything after the 'root' of the reduction,
    insert all instructions as close to their operands as possible. This
    can help reduce register pressure.

    Differential Revision: https://reviews.llvm.org/D67392 (detail)
    by sam_parker
  13. [HardwareLoops] Optimisation remarks

    This adds the initial plumbing to support optimisation remarks in
    the IR hardware-loop pass.

    I have left a todo in a comment where we can improve the reporting,
    and will iterate on that now that we have this initial support in.

    Differential Revision: https://reviews.llvm.org/D68579 (detail)
    by sjoerdmeijer
  14. [NFC] Replace a linked list in LiveDebugVariables pass with a DenseMap

    In LiveDebugVariables.cpp:
    Prior to this patch, UserValues were grouped into linked list chains. Each
    chain was the union of two sets: { A: Matching Source variable } or
    { B: Matching virtual register }. A ptr to the heads (or 'leaders')
    of each of these chains were kept in a map with the { Source variable } used
    as the key (set A predicate) and another with { Virtual register } as key
    (set B predicate).

    There was a search through the chains in the function getUserValue looking for
    UserValues with matching { Source variable, Complex expression, Inlined-at
    location }. Essentially searching for a subset of A through two interleaved
    linked lists of set A and B. Importantly, by design, the subset will only
    contain one or zero elements here. That is to say a UserValue can be uniquely
    identified by the tuple { Source variable, Complex expression, Inlined-at
    location } if it exists.

    This patch removes the linked list and instead uses a DenseMap to map
    the tuple { Source variable, Complex expression, Inlined-at location }
    to UserValue ptrs so that the getUserValue search predicate is this map key.
    The virtual register map now maps a vreg to a SmallVector<UserVal *> so that
    set B is still available for quick searches.

    Reviewers: aprantl, probinson, vsk, dblaikie

    Reviewed By: aprantl

    Subscribers: russell.gallop, gbedwell, bjope, hiraditya, llvm-commits

    Tags: #debug-info, #llvm

    Differential Revision: https://reviews.llvm.org/D68816 (detail)
    by orlandoch

Started by timer (4 times)

This run spent:

  • 3 hr 53 min waiting;
  • 4 hr 52 min build duration;
  • 8 hr 46 min total from scheduled to completion.

Identified problems

Regression test failed

This build failed because a regression test in the test suite FAILed. See the test report for details.
Indication 1

Ninja target failed

Below is a link to the first failed ninja target.
Indication 2

Missing test results

The test result file Jenkins is looking for does not exist after the build.
Indication 3