Started 10 mo ago
Took 24 min on green-dragon-24

Failed Build #56 (Dec 2, 2018 6:16:33 AM)

Build Artifacts
test_logs.tgz125 B view
Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 348090
  • http://llvm.org/svn/llvm-project/cfe/trunk : 348084
  • http://llvm.org/svn/llvm-project/lldb/trunk : 348058
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 347930
Changes
  1. [SelectionDAG] fold constant with undef vector per element

    This makes the SDAG behavior consistent with the way we do this in IR.
    It's possible that we were getting the wrong answer before. For example,
    'xor undef, undef --> 0' but 'xor undef, C' --> undef.

    But the most practical improvement is likely as shown in the tests here -
    for FP, we were overconstraining undef lanes to NaN, and that can prevent
    vector simplifications/narrowing (see D51553). (detail)
    by spatel
  2. [DAGCombiner] guard against an oversized shift crash

    This change prevents the crash noted in the post-commit comments
    for rL347478 :
    http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20181119/605166.html

    We can't guarantee that an oversized shift amount is folded away,
    so we have to check for it.

    Note that I committed an incomplete fix for that crash with:
    rL347502

    But as discussed here:
    http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20181126/605679.html
    ...we have to try harder.

    So I'm not sure how to expose the bug now (and apparently no fuzzers have found
    a way yet either).

    On the plus side, we have discovered that we're missing real optimizations by
    not simplifying nodes sooner, so the earlier fix still has value, and there's
    likely more value in extending that so we can simplify more opcodes and simplify
    when doing RAUW and/or putting nodes on the combiner worklist.

    Differential Revision: https://reviews.llvm.org/D54954 (detail)
    by spatel
  3. [ValueTracking] add helper function for testing implied condition; NFCI

    We were duplicating code around the existing isImpliedCondition() that
    checks for a predecessor block/dominating condition, so make that a
    wrapper call. (detail)
    by spatel

Started by timer (4 times)

This run spent:

  • 52 min waiting;
  • 24 min build duration;
  • 1 hr 17 min total from scheduled to completion.
Test Result (675 failures / ±0)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