Started 1 mo 5 days ago
Took 4 hr 26 min on green-dragon-02

Failed Build #14898 (Oct 16, 2019 5:47:39 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 375067
  • http://llvm.org/svn/llvm-project/cfe/trunk : 375063
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 374977
  • http://llvm.org/svn/llvm-project/zorg/trunk : 375036
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 374854
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 375058
Changes
  1. [gicombiner] Add the run-time rule disable option

    Summary:
    Each generated helper can be configured to generate an option that disables
    rules in that helper. This can be used to bisect rulesets.

    The disable bits are stored in a SparseVector as this is very cheap for the
    common case where nothing is disabled. It gets more expensive the more rules
    are disabled but you're generally doing that for debug purposes where
    performance is less of a concern.

    Depends on D68426

    Reviewers: volkan, bogner

    Reviewed By: volkan

    Subscribers: hiraditya, Petar.Avramovic, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68438 (detail)
    by dsanders
  2. [GISel][CombinerHelper] Add concat_vectors(build_vector, build_vector) => build_vector

    Teach the combiner helper how to flatten concat_vectors of build_vectors
    into a build_vector.

    Add this combine as part of AArch64 pre-legalizer combiner.

    Differential Revision: https://reviews.llvm.org/D69071 (detail)
    by qcolombet
  3. [lit] Improve lit.Run class

    * Push timing of overall test time into run module
    * Make lit.Run a proper class
    * Add a few TODO comments (detail)
    by yln
  4. [Concepts] ConceptSpecializationExprs mangling

    Implement mangling for CSEs to match regular template-ids.
    Reviewed as part of D41569 <https://reviews.llvm.org/D41569>.

    Re-commit fixing failing test. (detail)
    by saar.raz
  5. [lit] Fix another test case that r374652 missed (detail)
    by jdenny
  6. [gicombiner] Hoist pure C++ combine into the tablegen definition

    Summary:
    This is just moving the existing C++ code around and will be NFC w.r.t
    AArch64. Renamed 'CombineBr' to something more descriptive
    ('ElideByByInvertingCond') at the same time.

    The remaining combines in AArch64PreLegalizeCombiner require features that
    aren't implemented at this point and will be hoisted as they are added.

    Depends on D68424

    Reviewers: bogner, volkan

    Subscribers: kristof.beyls, hiraditya, Petar.Avramovic, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68426 (detail)
    by dsanders
  7. [lit] Remove unnecessary usage of lit.Run (detail)
    by yln
  8. [lit] Do not create semaphores when we do not need them

    Parallelism groups and semaphores are only required for parallel
    execution. (detail)
    by yln
  9. [lit] Factor out separate methods for parallel and serial execution (detail)
    by yln
  10. [NFC] Fix unused var in release builds (detail)
    by rupprecht
  11. Revert [support] GlobPattern: add support for `\` and `[!...]`, and allow `]` in more places

    This reverts r375051 (git commit a409afaad64ce83ea44cc30ee5f96b6e613a6e98)

    The patch does not work on Windows due to `\` in filenames being interpreted as escaping rather than literal path separators when used by lld linker scripts. (detail)
    by rupprecht
  12. [support] GlobPattern: add support for `\` and `[!...]`, and allow `]` in more places

    Summary: Update GlobPattern in libSupport to handle a few more cases. It does not fully match the `fnmatch` used by GNU objcopy since named character classes (e.g. `[[:digit:]]`) are not supported, but this should support most existing use cases (mostly just `*` is what's used anyway).

    This will be used to implement the `--wildcard` flag in llvm-objcopy to be more compatible with GNU objcopy.

    This is split off of D66613 to land the libSupport changes separately. The llvm-objcopy part will land soon.

    Reviewers: jhenderson, MaskRay, evgeny777, espindola, alexshap

    Reviewed By: MaskRay

    Subscribers: nickdesaulniers, emaste, arichardson, hiraditya, jakehehrlich, abrachet, seiya, llvm-commits

    Tags: #llvm

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

    undo objcopy changes to make this libsupport only (detail)
    by rupprecht
  13. [Utils] Cleanup similar cases to MergeBlockIntoPredecessor.

    Summary:
    There are two cases where a block is merged into its predecessor and the
    MergeBlockIntoPredecessor API is not used. Update the API so it can be
    reused in the other cases, in order to avoid code duplication.

    Cleanup motivated by D68659.

    Reviewers: chandlerc, sanjoy.google, george.burgess.iv

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68670 (detail)
    by asbirlea
  14. [lit] Print warning if we fail to delete temp directory (detail)
    by yln
  15. [lit] Skip creation of tmp dir if we don't actually run any tests (detail)
    by yln
  16. [lit] Remove return value from print_summary function (detail)
    by yln
  17. [lit] Small refactoring and cleanups in main.py

    * Remove outdated precautions for Python versions < 2.7
    * Remove dead code related to `maxIndividualTestTime` option
    * Move printing of test and result summary out of main into its own
      function

    Reviewed By: rnk

    Differential Revision: https://reviews.llvm.org/D68847 (detail)
    by yln
  18. Update ReleaseNotes: expand the section on enabling MemorySSA (detail)
    by asbirlea
  19. [dsymutil] Print warning/error for unknown/missing arguments.

    After changing dsymutil to use libOption, we lost error reporting for
    missing required arguments (input files). Additionally, we stopped
    complaining about unknown arguments. This patch fixes both and adds a
    test. (detail)
    by Jonas Devlieghere
  20. [AArch64] Fix offset calculation

    r374772 changed Offset to be an int64_t but left NewOffset as an int.
    Scale is unsigned, so in the calculation `Offset - NewOffset * Scale`,
    `NewOffset * Scale` was promoted to unsigned and was then zero-extended
    to 64 bits, leading to an incorrect computation which manifested as an
    out-of-memory when building the Swift standard library for Android
    aarch64. Promote NewOffset to int64_t to fix this, and promote
    EmittableOffset as well, since its one user passes it to a function
    which takes an int64_t anyway.

    Test case based on a suggestion by Sander de Smalen!

    Differential Revision: https://reviews.llvm.org/D69018 (detail)
    by smeenai
  21. GlobalISel: Implement lower for G_SADDO/G_SSUBO

    Port directly from SelectionDAG, minus the path using
    ISD::SADDSAT/ISD::SSUBSAT. (detail)
    by arsenm
  22. [Symbolize] Use the local MSVC C++ demangler instead of relying on dbghelp. NFC.

    This allows making a couple llvm-symbolizer tests run in all
    environments.

    Differential Revision: https://reviews.llvm.org/D68133 (detail)
    by mstorsjo
  23. Remove a stale comment, noted in post commit review for rL375038 (detail)
    by reames
  24. [clangd] Add the missing dependency on `clangLex`. (detail)
    by hliao

Started by timer (4 times)

This run spent:

  • 3 hr 36 min waiting;
  • 4 hr 26 min build duration;
  • 8 hr 3 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