Started 1 mo 0 days ago
Took 4 hr 27 min on green-dragon-02

Failed Build #14902 (Oct 17, 2019 12:11:54 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 375154
  • http://llvm.org/svn/llvm-project/cfe/trunk : 375134
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 375150
  • http://llvm.org/svn/llvm-project/zorg/trunk : 375141
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 374854
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 375117
Changes
  1. [Object] Fix the return type of getOffset/getSize

    Header64.offset/Header64.size are uint64_t, thus we should not
    truncate them to unit32_t. Moreover, there are a number of places
    where we sum the offset and the size (e.g. in various checks in MachOUniversal.cpp),
    the truncation causes issues since the offset/size can perfectly fit into uint32_t,
    while the sum overflows.

    Differential revision: https://reviews.llvm.org/D69126

    Test plan: make check-all (detail)
    by alexshap
  2. [NFC][InstCombine] Some more preparatory cleanup for dropRedundantMaskingOfLeftShiftInput() (detail)
    by lebedevri
  3. [PowerPC] Turn on CR-Logical reducer pass

    Quite a while ago, we implemented a pass that will reduce the number of
    CR-logical operations we emit. It does so by converting a CR-logical operation
    into a branch. We have kept this off by default because it seemed to cause a
    significant regression with one benchmark.
    However, that regression turned out to be due to a completely unrelated
    reason - AADB introducing a self-copy that is a priority-setting nop and it was
    just exacerbated by this pass.

    Now that we understand the reason for the only degradation, we can turn this
    pass on by default. We have long since fixed the cause for the degradation.

    Differential revision: https://reviews.llvm.org/D52431 (detail)
    by nemanjai
  4. [Builtins] Provide a mechanism to selectively disable tests based on whether an implementation is provided by a builtin library.

    Summary:
    If a platform removes some builtin implementations (e.g. via the
    Darwin-excludes mechanism) then this can lead to test failures because
    the test expects an implementation to be available.

    To solve this lit features are added for each configuration based
    on which sources are included in the builtin library. The features
    are of the form `librt_has_<name>` where `<name>` is the name of the
    source file with the file extension removed. This handles C and
    assembly sources.

    With the lit features in place it is possible to make certain tests
    require them.

    Example:

    ```
    REQUIRES: librt_has_comparedf2
    ```

    All top-level tests in `test/builtins/Unit` (i.e. not under
    `arm`, `ppc`, and `riscv`) have been annotated with the appropriate
    `REQUIRES: librt_has_*` statement.

    rdar://problem/55520987

    Reviewers: beanz, steven_wu, arphaman, dexonsmith, phosek, thakis

    Subscribers: mgorny, #sanitizers, llvm-commits

    Tags: #llvm, #sanitizers

    Differential Revision: https://reviews.llvm.org/D68064 (detail)
    by delcypher
  5. Reapply r375051: [support] GlobPattern: add support for `\` and `[!...]`, and allow `]` in more places

    Reland r375051 (reverted in r375052) after fixing lld tests on Windows in r375126 and r375131.

    Original description: 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 (detail)
    by rupprecht
  6. NFC: Fix variable only used in asserts by propagating the value.

    Summary:
    This fixes builds with assertions disabled that would otherwise
    fail with unused variable warnings

    Subscribers: nemanjai, hiraditya, kbarton, MaskRay, jsji, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69123 (detail)
    by saugustine
  7. [asan] Update Windows test expectations for LLVM's MS demangler

    After r375041 llvm-symbolizer uses it for demangling instead of
    UnDecorateSymbolName. LLVM puts spaces after commas while Microsoft does
    not. (detail)
    by rnk
  8. Revert [lit] Synthesize artificial deadline

    Python on Windows raises this OverflowError:
          gotit = waiter.acquire(True, timeout)
      OverflowError: timestamp too large to convert to C _PyTime_t

    So it seems this API behave the same way on every OS.

    Also reverts the dependent commit a660dc590a5e8dafa1ba6ed56447ede151d17bd9. (detail)
    by rnk
  9. [PowerPC] add tests for popcount with zext; NFC (detail)
    by spatel
  10. Updated UnifiedTreeBuilder to fully support is_legacy_mode.

    This allows to use the newly added LLVMBuildFactory ability to transparently use SVN or github and switch between depending on the is_legacy_mode attribute, and adds a github support to UnifiedTreeBuilder.

    Patch by Andrei Lebedev

    Differential Revision: https://reviews.llvm.org/D69049 (detail)
    by gkistanova
  11. Added support for git/github to LLVMBuildFactory.

    Now LLVMBuildFactory supports both a "legacy mode" with SVN checkout and a git/github checkout.

    This provides a transparent way of dialing with the source code in all the bots which use LLVMBuildFactory directly or indirectly through UnifiedTreeBuilder.

    is_legacy_mode is True by default to keep the status quo, but later will be changed to False to migrate all the supported bots to github.

    Patch by Andrei Lebedev

    Differential Revision: https://reviews.llvm.org/D69046 (detail)
    by gkistanova
  12. Reverted r375035 and r375036 to fix the commit message and author name. (detail)
    by gkistanova
  13. [IndVars] Split loop predication out of optimizeLoopExits [NFC]

    In the process of writing D69009, I realized we have two distinct sets of invariants within this single function, and basically no shared logic.  The optimize loop exit transforms (including the new one in D69009) only care about *analyzeable* exits.  Loop predication, on the other hand, has to reason about *all* exits.  At the moment, we have the property (due to the requirement for an exact btc) that all exits are analyzeable, but that will likely change in the future as we add widenable condition support. (detail)
    by reames
  14. [codeview] Workaround for PR43479, don't re-emit instr labels

    Summary:
    In the long run we should come up with another mechanism for marking
    call instructions as heap allocation sites, and remove this workaround.
    For now, we've had two bug reports about this, so let's apply this
    workaround. SLH (the other client of instruction labels) probably has
    the same bug, but the solution there is more likely to be to mark the
    call instruction as not duplicatable, which doesn't work for debug info.

    Reviewers: akhuang

    Subscribers: aprantl, hiraditya, aganea, chandlerc, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69068 (detail)
    by rnk
  15. Revert [Sanitizers] Add support for RISC-V 64-bit

    This reverts r375132 (git commit 00bbe990c5d4472d5413479a539b3d6edbb3ca7a) (detail)
    by lenary
  16. [NFC][InstCombine] Tests for "fold variable mask before variable shift-of-trunc" (PR42563)

    https://bugs.llvm.org/show_bug.cgi?id=42563 (detail)
    by lebedevri
  17. [OPENMP]Improve use of the global tid parameter.

    If we can determined, that the global tid parameter can be used in the
    function, better to use it rather than calling __kmpc_global_thread_num
    function. (detail)
    by abataev
  18. [IndVars] Factor out a helper function for readability [NFC] (detail)
    by reames
  19. [Sanitizers] Add support for RISC-V 64-bit

    Summary:
    This has been tested with gcc trunk on openSUSE Tumbleweed on the HiFive Unleashed.

    Patch by Andreas Schwab (schwab)

    Reviewers: luismarques

    Reviewed By: luismarques

    Subscribers: mhorne, emaste, luismarques, asb, mgorny, fedor.sergeev, simoncook, kito-cheng, shiva0217, rogfer01, rkruppe, lenary, s.egerton, #sanitizers, llvm-commits

    Tags: #llvm, #sanitizers

    Differential Revision: https://reviews.llvm.org/D66870 (detail)
    by lenary
  20. [lit] Move computation of deadline up into base class (detail)
    by yln
  21. [lit] Synthesize artificial deadline

    We always want to use a deadline when calling `result.await`.  Let's
    synthesize an artificial deadline (positive infinity) to simplify code
    and do less busy waiting. (detail)
    by yln
  22. [lit] Create derived classes for serial/parallel test runs

    The hope is that with a little OO we can nicely factor out the
    differences. (detail)
    by yln
  23. [ObjC] Diagnose implicit type coercion from ObjC 'Class' to object
    pointer types.

    For example, in Objective-C mode, the initialization of 'x' in:
    ```
      @implementation MyType
      + (void)someClassMethod {
        MyType *x = self;
      }
      @end
    ```
    is correctly diagnosed with an incompatible-pointer-types warning, but
    in Objective-C++ mode, it is not diagnosed at all -- even though
    incompatible pointer conversions generally become an error in C++.

    This patch fixes that oversight, allowing implicit conversions
    involving Class only to/from unqualified-id, and between qualified and
    unqualified Class, where the protocols are compatible.

    Note that this does change some behaviors in Objective-C, as well, as
    shown by the modified tests.

    Of particular note is that assignment from from 'Class<MyProtocol>' to
    'id<MyProtocol>' now warns. (Despite appearances, those are not
    compatible types. 'Class<MyProtocol>' is not expected to have instance
    methods defined by 'MyProtocol', while 'id<MyProtocol>' is.)

    Differential Revision: https://reviews.llvm.org/D67983 (detail)
    by jyknight
  24. [ObjC] Add some additional test cases around pointer conversions.

    This is especially important for Objective-C++, which is entirely
    missing this testing at the moment.

    This annotates with "FIXME" the cases which I change in the next
    patch -- I primarily wanted to document the current state of things so
    that the effect of the code change is made clear.

    Differential Revision: https://reviews.llvm.org/D67982 (detail)
    by jyknight

Started by timer (5 times)

This run spent:

  • 4 hr 1 min waiting;
  • 4 hr 27 min build duration;
  • 8 hr 28 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