Started 8 days 18 hr ago
Took 18 hr on green-dragon-09

Failed Build #5506 (Oct 7, 2019 3:44:27 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 373890
  • http://llvm.org/svn/llvm-project/cfe/trunk : 373887
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 373754
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 373809
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 373892
Changes
  1. [clangd] If an undocumented definition exists, don't accept documentation from other forward decls.

    Summary:
    This fixes cases like:
      foo.h
        class Undocumented{}
      bar.h
        // break an include cycle. we should refactor this!
        class Undocumented;
    Where the comment doesn't describe the class.

    Note that a forward decl that is *visible to the definition* will still have
    its doc comment used, by SymbolCollector: Merge isn't involved here.

    Reviewers: ilya-biryukov

    Subscribers: MaskRay, jkorous, arphaman, kadircet, usaxena95, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D68467 (detail/ViewSVN)
    by sammccall
  2. [llvm-readelf/llvm-objdump] - Improve/refactor the implementation of SHT_LLVM_ADDRSIG section dumping.

    This patch:

    * Adds a llvm-readobj/llvm-readelf test file for SHT_LLVM_ADDRSIG sections. (we do not have any)
    * Enables dumping of SHT_LLVM_ADDRSIG with --all.
    * Changes the logic to report a warning instead of an error when something goes wrong during dumping
      (allows to continue dumping SHT_LLVM_ADDRSIG and other sections on error).
    * Refactors a piece of logic to a new toULEB128Array helper which might be used for GNU-style
      dumping implementation.

    Differential revision: https://reviews.llvm.org/D68383 (detail/ViewSVN)
    by grimar
  3. [clangd] Collect missing macro references.

    Summary: Semantic highlghting is missing a few macro references.

    Reviewers: ilya-biryukov

    Subscribers: MaskRay, jkorous, arphaman, kadircet, usaxena95, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D68458 (detail/ViewSVN)
    by hokein
  4. [IA] Recognize hexadecimal escape sequences

    Summary:
    Implement support for hexadecimal escape sequences to match how GNU 'as'
    handles them. I.e., read all hexadecimal characters and truncate to the
    lower 16 bits.

    Reviewers: nickdesaulniers

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68483 (detail/ViewSVN)
    by void
  5. clang-cl: Ignore the new /ZH options

    These were added to the MS docs in
    https://github.com/MicrosoftDocs/cpp-docs/commit/85b9b6967e58e485251450f7451673f6fc873e88
    and are supposedly available in VS 2019 16.4 (though my 2019 Preview,
    version 16.4.0-pre.1.0 don't seem to have them.) (detail/ViewSVN)
    by hans
  6. [TableGen] Pacify gcc-5.4 more

    Followup to a previous pacification, this performs the same workaround
    to the TableGen generated code for tuple automata. (detail/ViewSVN)
    by jamesm
  7. Revert "[SLP] avoid reduction transform on patterns that the backend can load-combine"

    This reverts SVN r373833, as it caused a failed assert "Non-zero loop
    cost expected" on building numerous projects, see PR43582 for details
    and reproduction samples. (detail/ViewSVN)
    by mstorsjo
  8. [llvm-locstats] Fix a typo in the documentation; NFC (detail/ViewSVN)
    by djtodoro
  9. [X86] Autogenerate checks in leaFixup32.mir and leaFixup64.mir. NFC (detail/ViewSVN)
    by ctopper
  10. [X86] Support LEA64_32r in processInstrForSlow3OpLEA and use INC/DEC when possible.

    Move the erasing and iterator updating inside to match the
    other slow LEA function.

    I've adapted code from optTwoAddrLEA and basically rebuilt the
    implementation here. We do lose the kill flags now just like
    optTwoAddrLEA. This runs late enough in the pipeline that
    shouldn't really be a problem. (detail/ViewSVN)
    by ctopper
  11. [NFC][CGP] Tests for making ICMP_EQ use CR result of ICMP_S(L|G)T dominators (detail/ViewSVN)
    by yi-hong.lyu
  12. [c++20] Check for a class-specific operator delete when deleting an
    object of class type with a virtual destructor. (detail/ViewSVN)
    by rsmith
  13. Fix behavior of __builtin_bit_cast when the From and To types are the
    same.

    We were missing the lvalue-to-rvalue conversion entirely in this case,
    and in fact still need the full CK_LValueToRValueBitCast conversion to
    perform a load with no TBAA. (detail/ViewSVN)
    by rsmith
  14. gn build: no-op style tweak in sync script (detail/ViewSVN)
    by nico
  15. [Docs] Removes Subsystem Documentation page

    Removes Subsystem Documentation page. Also moves existing topics on Subsystem Documentation page to User Guides and Reference pages. (detail/ViewSVN)
    by dr87
  16. [X86][AVX] Access a scalar float/double as a free extract from a broadcast load (PR43217)

    If a fp scalar is loaded and then used as both a scalar and a vector broadcast, perform the load as a broadcast and then extract the scalar for 'free' from the 0th element.

    This involved switching the order of the X86ISD::BROADCAST combines so we only convert to X86ISD::BROADCAST_LOAD once all other canonicalizations have been attempted.

    Adds a DAGCombinerInfo::recursivelyDeleteUnusedNodes wrapper.

    Fixes PR43217

    Differential Revision: https://reviews.llvm.org/D68544 (detail/ViewSVN)
    by rksimon
  17. Fix signed/unsigned warning. NFCI (detail/ViewSVN)
    by rksimon
  18. [NFC][PowerPC] Reorganize CRNotPat multiclass patterns in PPCInstrInfo.td

    This is patch aims to group together the `CRNotPat` multi class instantiations
    within the `PPCInstrInfo.td` file.

    Integer instantiations of the multi class are grouped together into a section,
    and the floating point patterns are separated into its own section.

    Differential Revision: https://reviews.llvm.org/D67975 (detail/ViewSVN)
    by amyk
  19. [X86][SSE] Remove resolveTargetShuffleInputs and use getTargetShuffleInputs directly.

    Move the resolveTargetShuffleInputsAndMask call to after the shuffle mask combine before the undef/zero constant fold instead. (detail/ViewSVN)
    by rksimon
  20. [X86][SSE] Don't merge known undef/zero elements into target shuffle masks.

    Replaces setTargetShuffleZeroElements with getTargetShuffleAndZeroables which reports the Zeroable elements but doesn't merge them into the decoded target shuffle mask (the merging has been moved up into getTargetShuffleInputs until we can get rid of it entirely).

    This is part of the work to fix PR43024 and allow us to use SimplifyDemandedElts to simplify shuffle chains - we need to get to a point where the target shuffle mask isn't adjusted by its source inputs but instead we cache them in a parallel Zeroable mask. (detail/ViewSVN)
    by rksimon
  21. Implements CWG 1601 in [over.ics.rank/4.2]

    Summary:
    The overload resolution for enums with a fixed underlying type has changed in the C++14 standard. This patch implements the new rule.

    Patch by Mark de Wever!

    Reviewers: rsmith

    Reviewed By: rsmith

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D65695 (detail/ViewSVN)
    by rsmith
  22. [X86] Add custom type legalization for v16i64->v16i8 truncate and v8i64->v8i8 truncate when v8i64 isn't legal

    Summary:
    The default legalization for v16i64->v16i8 tries to create a multiple stage truncate concatenating after each stage and truncating again. But avx512 implements truncates with multiple uops. So it should be better to truncate all the way to the desired element size and then concatenate the pieces using unpckl instructions. This minimizes the number of 2 uop truncates. The unpcks are all single uop instructions.

    I tried to handle this by just custom splitting the v16i64->v16i8 shuffle. And hoped that the DAG combiner would leave the two halves in the state needed to make D68374 do the job for each half. This worked for the first half, but the second half got messed up. So I've implemented custom handling for v8i64->v8i8 when v8i64 needs to be split to produce the VTRUNCs directly.

    Reviewers: RKSimon, spatel

    Reviewed By: RKSimon

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68428 (detail/ViewSVN)
    by ctopper
  23. [LegalizeTypes][X86] When splitting a vselect for type legalization, don't split a setcc condition if the setcc input is legal and vXi1 conditions are supported

    Summary: The VSELECT splitting code tries to split a setcc input as well. But on avx512 where mask registers are well supported it should be better to just split the mask and use a single compare.

    Reviewers: RKSimon, spatel, efriedma

    Reviewed By: spatel

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68359 (detail/ViewSVN)
    by ctopper
  24. [Sema] Avoids an assertion failure when an invalid conversion declaration is used

    Summary:
    When using a user-defined conversion function template with a deduced return type the compiler gives a set of warnings:
    ```
    bug.cc:252:44: error: cannot specify any part of a return type in the declaration of a conversion function; use an alias template to declare a conversion to 'auto (Ts &&...) const'
      template <typename... Ts> operator auto()(Ts &&... xs) const;
                                               ^~~~~~~~~~~~~~~~~~~
    bug.cc:252:29: error: conversion function cannot convert to a function type
      template <typename... Ts> operator auto()(Ts &&... xs) const;
                                ^
    error: pointer to function type cannot have 'const' qualifier
    ```
    after which it triggers an assertion failure. It seems the last error is incorrect and doesn't have any location information. This patch stops the compilation after the second warning.

    Fixes bug 31422.

    Patch by Mark de Wever!

    Reviewers: rsmith

    Reviewed By: rsmith

    Subscribers: bbannier, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64820 (detail/ViewSVN)
    by rsmith
  25. gn build: make windows build less broken (detail/ViewSVN)
    by nico
  26. [LOOPGUARD] Remove asserts in getLoopGuardBranch
    Summary: The assertion in getLoopGuardBranch can be a 'return nullptr'
    under if condition.
    Authored By: DTharun
    Reviewer: Whitney, fhahn
    Reviewed By: Whitney, fhahn
    Subscribers: fhahn, llvm-commits
    Tag: LLVM
    Differential Revision: https://reviews.llvm.org/D66084 (detail/ViewSVN)
    by whitneyt
  27. [Docs] Removes Programming Documentation page

    Removes Programming Documentation page. Also moves existing topics on Programming Documentation page to User Guides and Reference pages. (detail/ViewSVN)
    by dr87

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18451
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18452
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18453
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18454
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18455
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18456
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18457
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18458
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18459
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18460
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18461
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18462
originally caused by:

This run spent:

  • 17 hr waiting;
  • 18 hr build duration;
  • 1 day 12 hr total from scheduled to completion.

Identified problems

Ninja target failed

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

Compile Error

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

Missing test results

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

Regression test failed

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