Started 21 days ago
Took 2 hr 22 min on green-dragon-18

Aborted Build rL:372893 - C:372889 - #777 (Sep 25, 2019 9:13:04 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 372893
  • http://llvm.org/svn/llvm-project/cfe/trunk : 372889
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 372849
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/zorg/trunk : 372433
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 372832
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 372888
Changes
  1. [CodeGen] Replace -max-jump-table-size with -max-jump-table-targets

    Modern processors predict the targets of an indirect branch regardless of
    the size of any jump table used to glean its target address.  Moreover,
    branch predictors typically use resources limited by the number of actual
    targets that occur at run time.

    This patch changes the semantics of the option `-max-jump-table-size` to limit
    the number of different targets instead of the number of entries in a jump
    table.  Thus, it is now renamed to `-max-jump-table-targets`.

    Before, when `-max-jump-table-size` was specified, it could happen that
    cluster jump tables could have targets used repeatedly, but each one was
    counted and typically resulted in tables with the same number of entries.
    With this patch, when specifying `-max-jump-table-targets`, tables may have
    different lengths, since the number of unique targets is counted towards the
    limit, but the number of unique targets in tables is the same, but for the
    last one containing the balance of targets.

    Differential revision: https://reviews.llvm.org/D60295 (detail/ViewSVN)
    by evandro
  2. [LangRef] Clarify absence of rounding guarantees for fmuladd.

    During the review of D67434, it was recommended to make fmuladd's
    behavior more explicit. D67434 depends on this interpretation.

    Reviewers: efriedma, jfb, reames, scanon, lebedev.ri, spatel

    Reviewed By: spatel

    Differential Revision: https://reviews.llvm.org/D67552 (detail/ViewSVN)
    by fhahn
  3. Re-land r372863: [AST] Extract Decl::printNestedNameSpecifier helper from Decl::printQualifiedName

    Reverted in r372880 due to the test failure.
    Also contains a fix that adjusts printQualifiedName to return the same results as before in
    case of anonymous function locals and parameters. (detail/ViewSVN)
    by ibiryukov
  4. [clangd] Change constness of parameters to findExplicitRefs

    Summary:
    Recursive AST requires non-const ast nodes, but it doesn't really
    mutate them. In addition to that, in clangd we mostly have const ast nodes. So
    it makes sense to move the const_cast into callee rather than having it at every
    caller in the future.

    Reviewers: ilya-biryukov

    Subscribers: MaskRay, jkorous, arphaman, usaxena95, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D68027 (detail/ViewSVN)
    by kadircet
  5. [DAGCombiner] add one-use restriction to vector transform with cheap extract

    We might be able to do better on the example in the test,
    but in general, we should not scalarize a splatted vector
    binop if there are other uses of the binop. Otherwise, we
    can end up with code as we had - a scalar op that is
    redundant with a vector op. (detail/ViewSVN)
    by spatel
  6. [PatternMatch] Make m_Br more flexible, add matchers for BB values.

    Currently m_Br only takes references to BasicBlock*, which limits its
    flexibility. For example, you have to declare a variable, even if you
    ignore the result or you have to have additional checks to make sure the
    matched BB matches an expected one.

    This patch adds m_BasicBlock and m_SpecificBB matchers, which can be
    used like the existing matchers for constants or values.

    I also had a look at the existing uses and updated a few. IMO it makes
    the code a bit more explicit.

    Reviewers: spatel, craig.topper, RKSimon, majnemer, lebedev.ri

    Reviewed By: lebedev.ri

    Differential Revision: https://reviews.llvm.org/D68013 (detail/ViewSVN)
    by fhahn
  7. [x86] add test for multi-use scalarization of vector binop; NFC (detail/ViewSVN)
    by spatel
  8. [TargetInstrInfo] Let findCommutedOpIndices take const MachineInstr&

    Neither the base implementation of findCommutedOpIndices nor any in-tree target modifies the instruction passed in and there is no reason why they would in the future.

    Committed on behalf of @hvdijk (Harald van Dijk)

    Differential Revision: https://reviews.llvm.org/D66138 (detail/ViewSVN)
    by rksimon
  9. [PatternMatch] Generalize brc_match (NFC).

    Preparation for D68013. (detail/ViewSVN)
    by fhahn

Started by an SCM change (8 times)

This run spent:

  • 1 hr 21 min waiting;
  • 2 hr 22 min build duration;
  • 3 hr 43 min total from scheduled to completion.

Aborted by user Azharuddin Mohammed

Identified problems

Missing test results

The test result file Jenkins is looking for does not exist after the build.
Indication 1