Started 11 days ago
Took 4 hr 54 min on green-dragon-02

Failed Build #14379 (Jul 12, 2019 11:39:20 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 365938
  • http://llvm.org/svn/llvm-project/cfe/trunk : 365935
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 365880
  • http://llvm.org/svn/llvm-project/zorg/trunk : 365596
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 365923
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 365937
Changes
  1. Addition to rL365925, removing remaining virtuals (detail)
    by sstefan
  2. [clang-doc] Add html links to references

    <a> tags are added for the parents and members of records and return type and
    params of functions. The link redirects to the reference's info file.

    The directory path where each info file will be saved is now generated in the
    serialization phase and stored as an attribute in each Info.

    Bitcode writer and reader were modified to handle the new attributes.

    Committed on behalf of Diego Astiazarán (diegoaat97@gmail.com).

    Differential Revision: https://reviews.llvm.org/D63663 (detail)
    by juliehockett
  3. Slightly simplify MappedBlockStream::createIndexedStream() calls

    All callers had a PDBFile object at hand, so call
    Pdb.createIndexedStream() instead, which pre-populates all the arguments
    (and returns nullptr for kInvalidStreamIndex).

    Also change safelyCreateIndexedStream() to only take the string index,
    and update callers. Make the method public and call it in two places
    that manually did the bounds checking before.

    No intended behavior change.

    Differential Revision: https://reviews.llvm.org/D64633 (detail)
    by nico
  4. [WebAssembly] Make pthread imply bulk-memory, mutable-globals

    Summary:
    This paves the way for using passive segments in pthread builds, which
    will make separate memory files unnecessary. Mutable globals are also
    necessary for the upcoming implementation of TLS.

    Reviewers: aheejin, dschuff

    Subscribers: sbc100, jgravelle-google, sunfish, jfb, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64586 (detail)
    by tlively
  5. Add explicit newline at end of `llvm-pdbutil dump`

    All dump modes I checked didn't print a trailing newline, so add one. (detail)
    by nico
  6. [SystemZ] Add support for new cpu architecture - arch13

    This patch series adds support for the next-generation arch13
    CPU architecture to the SystemZ backend.

    This includes:
    - Basic support for the new processor and its features.
    - Support for low-level builtins mapped to new LLVM intrinsics.
    - New high-level intrinsics in vecintrin.h.
    - Indicate support by defining  __VEC__ == 10303.

    Note: No currently available Z system supports the arch13
    architecture.  Once new systems become available, the
    official system name will be added as supported -march name. (detail)
    by uweigand
  7. [SystemZ] Add support for new cpu architecture - arch13

    This patch series adds support for the next-generation arch13
    CPU architecture to the SystemZ backend.

    This includes:
    - Basic support for the new processor and its features.
    - Assembler/disassembler support for new instructions.
    - CodeGen for new instructions, including new LLVM intrinsics.
    - Scheduler description for the new processor.
    - Detection of arch13 as host processor.

    Note: No currently available Z system supports the arch13
    architecture.  Once new systems become available, the
    official system name will be added as supported -march name. (detail)
    by uweigand
  8. Remove unused methods in Sancov. (detail)
    by leonardchan
  9. [X86] Add NEG to isUseDefConvertible.

    We can use the C flag from NEG to detect that the input was zero.

    Really we could probably use the Z flag too. But C matches what
    we'd do for usubo 0, X.

    Haven't found a test case for this due to the usubo formation
    in CGP. But I verified if I comment out the CGP code this
    transformation catches some of the same cases. (detail)
    by ctopper
  10. [X86][AVX] Add PR34359 shuffle test case. (detail)
    by rksimon
  11. [Attributor] Removing unnecessary `virtual` keywords.

    Some function in the Attributor framework are unnecessarily
    marked virtual. This patch removes virtual keyword

    Reviewers: jdoerfert

    Subscribers: hiraditya, llvm-commits

    Differential Revision: https://reviews.llvm.org/D64637 (detail)
    by sstefan
  12. [Attributor] Deduce "nofree" function attribute

    Summary: Deduce "nofree" function attribute. A more concise description of "nofree" is on D49165.

    Reviewers: jdoerfert

    Reviewed By: jdoerfert

    Subscribers: homerdin, hfinkel, lebedev.ri, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D62687 (detail)
    by uenoku
  13. [libc++] Add XFAILs for CTAD tests on older compilers (detail)
    by Louis Dionne
  14. Revert [clang-shlib] Fix clang-shlib for PRIVATE dependencies

    This reverts r365825 (git commit 3173c60f96c3ccfc17d403a192ae58e720153c23)

    This is breaking BUILD_SHARED_LIBS=ON builds. Reverting while I rethink it. (detail)
    by smeenai
  15. CodeGet: Init 32bit pointers with 0xFFFFFFFF

    Summary:
    Patch makes D63967 effective for 32bit platforms and improves pattern
    initialization there. It cuts size of 32bit binary compiled with
    -ftrivial-auto-var-init=pattern by 2% (3% with -Os).

    Binary size change on CTMark, (with -fuse-ld=lld -Wl,--icf=all, similar results with default linker options)
    ```
                       master           patch      diff
    Os pattern   7.915580e+05    7.698424e+05 -0.028387
    O3 pattern   9.953688e+05    9.752952e+05 -0.019325
    ```

    Zero vs Pattern on master
    ```
                   zero       pattern      diff
    Os     7.689712e+05  7.915580e+05  0.031380
    O3     9.744796e+05  9.953688e+05  0.021133
    ```

    Zero vs Pattern with the patch
    ```
                   zero       pattern      diff
    Os     7.689712e+05  7.698424e+05  0.000789
    O3     9.744796e+05  9.752952e+05  0.000742
    ```

    Reviewers: pcc, eugenis, glider, jfb

    Reviewed By: jfb

    Subscribers: hubert.reinterpretcast, dexonsmith, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64597 (detail)
    by Vitaly Buka
  16. [IndVars] Use exit count reasoning to discharge obviously untaken exits

    Continue in the spirit of D63618, and use exit count reasoning to prove away loop exits which can not be taken since the backedge taken count of the loop as a whole is provably less than the minimal BE count required to take this particular loop exit.

    As demonstrated in the newly added tests, this triggers in a number of cases where IndVars was previously unable to discharge obviously redundant exit tests. And some not so obvious ones.

    Differential Revision: https://reviews.llvm.org/D63733 (detail)
    by reames
  17. Dump actual line numbers when dumping the AST to JSON.

    The "line" attribute is now the physical line within the source file for the location. A "presumedLine" attribute is printed when the presumed line number does not match the given source line number. We continue to not print repeated line information in subsequent source locations, but we track presumed and actual lines separately. (detail)
    by aaronballman
  18. Fix some minor coding-style issues in git-llvm. (detail)
    by jyknight
  19. Allow the 'git-llvm' tool to push to svn from the split repositories
    for 'test-suite', 'lnt', 'zorg', and 'www'.

    This is done by looking at the pathname of the git remote named
    'origin', which is not 100% reliable, but should work in most cases. (detail)
    by jyknight
  20. [FunctionAttrs] Add a test for "nofree" function attribute

    This patch adds a test for nofree function attribute. (detail)
    by uenoku
  21. [Support] Move the static initializer install_out_memory_new_handler to InitLLVM

    An application linking against LLVMSupport should not get the gratuitous
    set::std_new_handler call.

    Reviewed By: jfb

    Differential Revision: https://reviews.llvm.org/D64505 (detail)
    by maskray
  22. Add missing <atomic> include to appease MSVC builds. (detail)
    by rksimon
  23. Minor cleanup.

    Simplify things a bit by removing unnecessary full type qualification.
    This also happens to avoid a build break with now-unsupported
    Visual Studio 2015.

    Differential Review: https://reviews.llvm.org/D64588 (detail)
    by tra
  24. Test commit (detail)
    by uenoku
  25. Support for dumping current PrettyStackTrace on SIGINFO (Ctrl-T)

    Support SIGINFO (and SIGUSR1 for POSIX purposes) to tell what
    long-running jobs are doing, as inspired by BSD tools (including on
    macOS), by dumping the current PrettyStackTrace.

    This adds a new kind of signal handler for non-fatal "info" signals,
    similar to the "interrupt" handler that already exists for SIGINT
    (Ctrl-C). It then uses that handler to update a "generation count"
    managed by the PrettyStackTrace infrastructure, which is then checked
    whenever a PrettyStackTraceEntry is pushed or popped on each
    thread. If the generation has changed---i.e. if the user has pressed
    Ctrl-T---the stack trace is dumped, though unfortunately it can't
    include the deepest entry because that one is currently being
    constructed/destructed.

    https://reviews.llvm.org/D63750 (detail)
    by jrose
  26. [AMDGPU] Fix DPP combiner check for exec modification

    Summary:
    r363675 changed the exec modification helper function, now called
    execMayBeModifiedBeforeUse, so that if no UseMI is specified it checks
    all instructions in the basic block, even beyond the last use. That
    meant that the DPP combiner no longer worked in any basic block that
    ended with a control flow instruction, and in particular it didn't work
    on code sequences generated by the atomic optimizer.

    Fix it by reinstating the old behaviour but in a new helper function
    execMayBeModifiedBeforeAnyUse, and limiting the number of instructions
    scanned.

    Reviewers: arsenm, vpykhtin

    Subscribers: kzhuravl, nemanjai, jvesely, wdng, nhaehnle, yaxunl, dstuttard, tpr, t-tye, hiraditya, kbarton, MaskRay, jfb, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64393 (detail)
    by foad
  27. [clang-format][tests] Explicitly specify style in some tests

    Summary: This fixes broken tests when doing an out-of-source build that picks up a random .clang-format on the file system due to the default "file" style.

    Reviewers: djasper, klimek, MyDeveloperDay, krasimir

    Reviewed By: MyDeveloperDay

    Subscribers: lebedev.ri, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D61001 (detail)
    by r.stahl
  28. [LegacyPassManager] Small ModuleCount cleanup (detail)
    by maskray
  29. Revert "[JSONCompilationDatabase] Strip distcc/ccache/gomacc wrappers from parsed commands."

    New test is failing on Windows bot

    This reverts commit 9c0391b36a76f8e3949588de3f44b7314c2318bf. (detail)
    by russell_gallop
  30. cmake: Fix install of libclang-cpp.so when LLVM_INSTALL_TOOLCHAIN_ONLY=ON

    Summary:
    If CLANG_LINK_CLANG_DYLIB is also enabled, then this library needs to be
    installed.

    Fixes PR42575.

    Reviewers: beanz, smeenai

    Subscribers: mgorny, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64582

    Conflicts:
    clang/tools/clang-shlib/CMakeLists.txt (detail)
    by tstellar
  31. [AMDGPU] Restrict v_cndmask_b32 abs/neg modifiers to f32

    Summary:
    D64497 allowed abs/neg source modifiers on v_cndmask_b32 but it doesn't
    make any sense to apply them to f16 operands; they would interpret the
    bits of the value as an f32, giving nonsensical results. This patch
    restricts them to f32 operands.

    Reviewers: arsenm, hakzsam

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

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64636 (detail)
    by foad
  32. Delete dead stores (detail)
    by maskray
  33. cmake: Add INSTALL_WITH_TOOLCHAIN option to add_*_library macros

    Summary:
    This will simplify the macros by allowing us to remove the hard-coded
    list of libraries that should be installed when
    LLVM_INSTALL_TOOLCHAIN_ONLY is enabled.

    Reviewers: beanz, smeenai

    Reviewed By: beanz

    Subscribers: aheejin, mehdi_amini, mgorny, steven_wu, dexonsmith, cfe-commits, llvm-commits

    Tags: #clang, #llvm

    Differential Revision: https://reviews.llvm.org/D64580 (detail)
    by tstellar
  34. Delete dead stores (detail)
    by maskray
  35. Dump floating-point values as strings when dumping to JSON.

    This fixes a bug where we would have an invalid JSON attribute (e.g., "value": inf). It also increases the precision of the values because they're not represented as approximate doubles with the host architecture's floating-point model. (detail)
    by aaronballman

Started by timer (5 times)

This run spent:

  • 4 hr 28 min waiting;
  • 4 hr 54 min build duration;
  • 9 hr 23 min total from scheduled to completion.
Test Result (3 failures / -2)Show all failed tests >>>

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

Compile Error

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