Started 8 mo 8 days ago
Took 1 hr 39 min on green-dragon-14

Failed Build #294 (Feb 15, 2019 7:44:47 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 354139
  • http://llvm.org/svn/llvm-project/cfe/trunk : 354141
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 354143
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 346271
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 353955
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 354116
Changes
  1. [compiler-rt] Let CMake search for thread support

    This is an educated guess to fix sanitizer-x86_64-linux after r354132. (detail/ViewSVN)
    by hahnfeld
  2. [clang] Add build and install targets for clang libraries

    This is modeled after the existing llvm-libraries target. It's a
    convenient way to include all clang libraries in a distribution.

    This differs slightly from the llvm-libraries target in that it adds any
    library added via add_clang_library, whereas llvm-libraries only
    includes targets added via add_llvm_library that didn't use the MODULE
    or BUILDTREE_ONLY arguments. add_clang_library doesn't appear to have
    any equivalents of those arguments, so the conditions don't apply.

    Differential Revision: https://reviews.llvm.org/D58269 (detail/ViewSVN)
    by smeenai
  3. [clang] Create install targets for non-shared libraries

    I don't see a reason for these to not have install targets created,
    which in turn allows them to be bundled in distributions. This doesn't
    affect the "install" target, since that just runs all CMake install
    rules (and we were already creating install rules for these).

    Differential Revision: https://reviews.llvm.org/D58268 (detail/ViewSVN)
    by smeenai
  4. GlobalISel: Fix inadequate verification of g_build_vector

    Testing based on the total size of the elements failed to catch a few
    invalid scenarios, so explicitly check the number of elements/operands
    and types.

    This failed to catch situations like
    <4 x s16> = G_BUILD_VECTOR s32, s32 since the total size added
    up. This also would fail to catch an implicit conversion between
    pointers and scalars. (detail/ViewSVN)
    by arsenm
  5. Try to organize MachineVerifier tests

    The Verifier is separate from the MachineVerifier, so move it to a
    different directory. Some other verifier tests were scattered in
    target codegen tests as well (although I'm sure I missed some). Work
    towards using a more consistent naming scheme to make it clearer where
    the gaps still are for generic instructions. (detail/ViewSVN)
    by arsenm
  6. OptionalStorage implementation for trivial type, take III

    This is another attempt at implementating optional storage
    for trivially copyable type, using an union instead of a
    raw buffer to hold the actual storage. This make it possible
    to get rid of the reinterpret_cast, and hopefully to fix the UB
    of the previous attempts.

    This validates fine on my laptop for gcc 8.2 and gcc 4.8, I'll
    revert if it breaks the validation. (detail/ViewSVN)
    by serge_sans_paille
  7. Exteded test of . (detail/ViewSVN)
    by ymandel
  8. Remove unnecessary expectation. (detail/ViewSVN)
    by ymandel
  9. Added test for matcher On. (detail/ViewSVN)
    by ymandel
  10. Add tests for assorted `CXXMemberCallExpr` matchers.

    Summary: Add tests for matchers `on`, `onImplicitObjectArgument` and `hasObjectExpression`.

    Reviewers: alexfh, steveire, aaron.ballman

    Differential Revision: https://reviews.llvm.org/D56850 (detail/ViewSVN)
    by ymandel
  11. [compiler-rt] Cleanup usage of C++ ABI library

    Add missed value "libcxxabi" and introduce SANITIZER_TEST_CXX for linking
    unit tests. This needs to be a full C++ library and cannot be libcxxabi.

    Differential Revision: https://reviews.llvm.org/D58012 (detail/ViewSVN)
    by hahnfeld
  12. [MergeICmps] Make base ordering really deterministic.

    Summary:
    The idea is that we now manipulate bases through a `unsigned BaseID` based on
    order of appearance in the comparison chain rather than through the `Value*`.

    Fixes 40714.

    Reviewers: gchatelet

    Subscribers: mgrang, jfb, jdoerfert, llvm-commits, hans

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D58274 (detail/ViewSVN)
    by courbet
  13. [MergeICmps][NFC] Improve doc. (detail/ViewSVN)
    by courbet
  14. [Analyzer] Fix for test file of bug 40625

    Test fixed and changed to true positive, FIXME about false positive removed. (detail/ViewSVN)
    by baloghadamsoftware
  15. Speculatively revert r354051 "Recommit Optional specialization for trivially copyable types"
    and
    r354055 "Optional specialization for trivially copyable types, part2"

    These are suspected to cause Clang to get miscompiled on Ubuntu 14.04
    (Trusty) which uses GCC 4.8.4. Reverting for an hour to see if this
    helps. See llvm-commits thread.

    > Recommit Optional specialization for trivially copyable types
    >
    > Unfortunately the original code gets misscompiled by GCC (at least 8.1),
    > this is a tentative workaround using std::memcpy instead of inplace new
    > for trivially copyable types. I'll revert if it breaks.
    >
    > Original revision: https://reviews.llvm.org/D57097 (detail/ViewSVN)
    by hans
  16. [NFCI] Factor out block removal from stack of nested loops (detail/ViewSVN)
    by mkazantsev
  17. Fix "field 'DFS' will be initialized after field 'DTU'" warning. NFCI. (detail/ViewSVN)
    by rksimon
  18. [OpenCL][PR40707] Allow OpenCL C types in C++ mode.

    Allow all OpenCL types to be parsed in C++ mode. (detail/ViewSVN)
    by stulova
  19. [ASTImporter] Import every Decl in lambda record

    Summary:
    Previously only the fields were imported. Now every Decl is imported.
    This way the destructor decl is not missing after import.

    Patch by balazske (Balázs Kéri)

    Reviewers: a.sidorin, shafik

    Reviewed By: shafik

    Subscribers: balazske, cfe-commits, Szelethus, martong, dkrupp

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D57740 (detail/ViewSVN)
    by martong
  20. [BPI] Look through bitcasts in calcZeroHeuristic

    Constant hoisting may have hidden a constant behind a bitcast so that
    it isn't folded into its users. However, this prevents BPI from
    calculating some of its heuristics that are based upon constant
    values. So, I've added a simple helper function to look through these
    casts.

    Differential Revision: https://reviews.llvm.org/D58166 (detail/ViewSVN)
    by sam_parker
  21. [NFC] Promote DFS to field for further use (detail/ViewSVN)
    by mkazantsev
  22. [X86][AVX] lowerShuffleAsLanePermuteAndPermute - fully populate the lane shuffle mask (PR40730)

    As detailed on PR40730, we are not correctly filling in the lane shuffle mask (D53148/rL344446) - we fill in for the correct src lane but don't add it to the correct mask element, so any reference to the correct element is likely to see an UNDEF mask index.

    This allows constant folding to propagate UNDEFs prior to the lane mask being (correctly) lowered to vperm2f128.

    This patch fixes the issue by fully populating the lane shuffle mask - this is more than is necessary (if we only filled in the required mask elements we might be able to match other shuffle instructions - broadcasts etc.), but its the most cautious approach as this needs to be cherrypicked into the 8.0.0 release branch.

    Differential Revision: https://reviews.llvm.org/D58237 (detail/ViewSVN)
    by rksimon
  23. [clangd] Unlink VFS working dir from OS working dir. Reland of r351051 (detail/ViewSVN)
    by sammccall
  24. [ARM GlobalISel] Style fix. NFCI

    Add the opcode for ADDrr / t2ADDrr to the Opcode cache, as we did for
    all other opcodes where the handling is otherwise the same between arm
    mode and thumb2. (detail/ViewSVN)
    by rovka
  25. [ARM GlobalISel] Support branches for Thumb2

    Just like arm mode, but with different opcodes. (detail/ViewSVN)
    by rovka
  26. [RISCV] Add assembler support for LA pseudo-instruction

    This patch also introduces the emitAuipcInstPair helper, which is then used
    for both emitLoadAddress and emitLoadLocalAddress.

    Differential Revision: https://reviews.llvm.org/D55325
    Patch by James Clarke. (detail/ViewSVN)
    by asb
  27. [RISCV] Support assembling %got_pcrel_hi operator

    Differential Revision: https://reviews.llvm.org/D55279
    Patch by James Clarke. (detail/ViewSVN)
    by asb
  28. Revert "[Analysis] -Wunreachable-code shouldn't fire on the increment of a foreach loop"

    This reverts commit r354102. (detail/ViewSVN)
    by sammccall
  29. [ARM CGP] Fix ConvertTruncs

    ConvertTruncs is used to replace a trunc for an AND mask, however
    this function wasn't working as expected. By performing the change
    later, we can create a wide type integer mask instead of a narrow -1
    value, which could then be simply removed (incorrectly). Because we
    now perform this action later, it's necessary to cache the trunc type
    before we perform the promotion.

    Differential Revision: https://reviews.llvm.org/D57686 (detail/ViewSVN)
    by sam_parker
  30. [NFC] Tweak SplitBlockAndInsertIfThen to use existing ThenBlock (detail/ViewSVN)
    by mkazantsev
  31. [Analysis] -Wunreachable-code shouldn't fire on the increment of a foreach loop

    Summary:
    The idea is that the code here isn't written, so doesn't indicate a bug.
    Similar to code expanded from macros.

    This means the warning no longer fires on this code:
      for (auto C : collection) {
        process(C);
        return;
      }
      handleEmptyCollection();
    Unclear whether this is more often a bug or not in practice, I think it's a
    reasonable idiom in some cases.
    Either way, if we want to warn on "loop that doesn't loop", I think it should be
    a separate warning, and catch `while(1) break;`

    Reviewers: ilya-biryukov, ioeric

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D58134 (detail/ViewSVN)
    by sammccall
  32. [TEST] Update test comments, refactor checks with update_test_checks.py

    This patch changes messages in guards-related tests to adequately reflect
    reality. (detail/ViewSVN)
    by mkazantsev

Started by upstream project Clang Stage 2: cmake, R -g Asan&UBSan, using Stage 1 RA, Phase 1 build number 6137
originally caused by:

This run spent:

  • 1 min 5 sec waiting;
  • 1 hr 39 min build duration;
  • 1 hr 40 min total from scheduled to completion.

Identified problems

Missing test results

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

CMake Error

This build failed because of an CMake configuration error. Below is a list of all errors in the build log:
Indication 2