Started 7 mo 7 days ago
Took 39 min

Failed Build clang-r359997-t56182-b56182.tar.gz (May 5, 2019 1:22:08 PM)

Issues

Found 1 issues:
Error: <html>
+ . /Users/buildslave/jenkins/workspace/lnt-test-suite-x86_64-Os-flto/config/tasks/utils/lnt_submit.sh
++ echo '@@@ LNT Submit @@@'
@@@ LNT Submit @@@
++ '[' -n http://104.154.54.203/db_default/v4/nts/submitRun -a -n lnt-test-suite-x86_64-Os-flto ']'
+++ lnt submit http://104.154.54.203/db_default/v4/nts/submitRun /Users/buildslave/jenkins/workspace/lnt-test-suite-x86_64-Os-flto/lnt-sandbox/report.json
++ LNT_RESULT_URL='error: <html>

<head><title>504 Gateway Time-out</title></head>
<body bgcolor="white">

Build Log

Revision: 358206
Changes
  1. [NFC] BasicBlock: generalize replaceSuccessorsPhiUsesWith(), take Old bb

    Thus it does not assume that the old basic block is the basic block
    for which we are looking at successors.

    Not reviewed, but seems rather trivial, in line with the rest of
    previous few patches. (detail)
    by lebedevri
  2. [NFC] BasicBlock: refactor changePhiUses() out of replacePhiUsesWith(), use it

    Summary:
    It is a common thing to loop over every `PHINode` in some `BasicBlock`
    and change old `BasicBlock` incoming block to a new `BasicBlock` incoming block.
    `replaceSuccessorsPhiUsesWith()` already had code to do that,
    it just wasn't a function.
    So outline it into a new function, and use it.

    Reviewers: chandlerc, craig.topper, spatel, danielcdh

    Reviewed By: craig.topper

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D61013 (detail)
    by lebedevri
  3. [NFC] PHINode: introduce replaceIncomingBlockWith() function, use it

    Summary:
    There is `PHINode::getBasicBlockIndex()`, `PHINode::setIncomingBlock()`
    and `PHINode::getNumOperands()`, but no function to replace every
    specified `BasicBlock*` predecessor with some other specified `BasicBlock*`.
    Clearly, there are a lot of places that could use that functionality.

    Reviewers: chandlerc, craig.topper, spatel, danielcdh

    Reviewed By: craig.topper

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D61011 (detail)
    by lebedevri
  4. [NFC] Instruction: introduce replaceSuccessorWith() function, use it

    Summary:
    There is `Instruction::getNumSuccessors()`, `Instruction::getSuccessor()`
    and `Instruction::setSuccessor()`, but no function to replace every
    specified `BasicBlock*` successor with some other specified `BasicBlock*`.
    I've found one place where it should clearly be used.

    Reviewers: chandlerc, craig.topper, spatel, danielcdh

    Reviewed By: craig.topper

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D61010 (detail)
    by lebedevri
  5. [NFC][Utils] deleteDeadLoop(): add an assert that exit block has some non-PHI instruction

    Summary:
    If `deleteDeadLoop()` is called on such a loop, that has "bad" exit block,
    one that e.g. has no terminator instruction, the `DIBuilder::insertDbgValueIntrinsic()`
    will be told to insert the Dbg Value Intrinsic after `nullptr`
    (since there is no first non-PHI instruction), which will cause it to not insert
    those instructions into any basic block. The instructions will be parent-less,
    and IR verifier will complain. It is rather obvious to track down the root cause
    when that happens, so let's just assert it never happens.

    Reviewers: sanjoy, davide, vsk

    Reviewed By: vsk

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D61008 (detail)
    by lebedevri

Started by upstream project relay-lnt-test-suite build number 7205
originally caused by:

This run spent:

  • 9.6 sec waiting;
  • 39 min build duration;
  • 39 min total from scheduled to completion.

Identified problems

No identified problem

No problems were identified. If you know why this problem occurred, please add a suitable Cause for it.