Started 29 days ago
Took 1 hr 20 min on green-dragon-17

Success Build rL:372275 - C:372269 - #665 (Sep 18, 2019 4:22:52 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 372275
  • http://llvm.org/svn/llvm-project/cfe/trunk : 372269
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 372205
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/zorg/trunk : 371835
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 372242
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 372206
Changes
  1. [WebAssembly] Restore defaults for stores per memop

    Summary:
    Large slowdowns were observed in Rust due to many small, constant
    sized copies in conjunction with poorly-optimized memory.copy
    implementations. Since memory.copy cannot be expected to be inlined
    efficiently by engines at this time, stop using it for the smallest
    copies. We continue to lower all memcpy intrinsics to memory.copy,
    though.

    Reviewers: aheejin, alexcrichton

    Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, JDevlieghere, sunfish, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D67639 (detail/ViewSVN)
    by tlively
  2. [Docs] Moves topics to new categories

    This commit moves several topics to new categories. It also removes a few duplicate links in Subsystem Documentation. (detail/ViewSVN)
    by dr87
  3. [AArch64][GlobalISel] Support lowering musttail calls

    Since we now lower most tail calls, it makes sense to support musttail.

    Instead of always falling back to SelectionDAG, only fall back when a musttail
    call was not able to be emitted as a tail call. Once we can handle most
    incoming and outgoing arguments, we can change this to a `report_fatal_error`
    like in ISelLowering.

    Remove the assert that we don't have varargs and a musttail, and replace it
    with a return false. Implementing this requires that we implement
    `saveVarArgRegisters` from AArch64ISelLowering, which is an entirely different
    patch.

    Add GlobalISel lines to vararg-tallcall.ll to make sure that we produce correct
    code. Right now we only fall back, but eventually this will be relevant.

    Differential Revision: https://reviews.llvm.org/D67681 (detail/ViewSVN)
    by paquette
  4. Remove the obsolete BlockByRefStruct flag from LLVM IR

    DIFlagBlockByRefStruct is an unused DIFlag that originally was used by
    clang to express (Objective-)C block captures in debug info. For the
    last year Clang has been emitting complex DIExpressions to describe
    block captures instead, which makes all the code supporting this flag
    redundant.

    This patch removes the flag and all supporting "dead" code, so we can
    reuse the bit for something else in the future.

    Since this only affects debug info generated by Clang with the block
    extension this mostly affects Apple platforms and I don't have any
    bitcode compatibility concerns for removing this. The Verifier will
    reject debug info that uses the bit and thus degrade gracefully when
    LTO'ing older bitcode with a newer compiler.

    rdar://problem/44304813

    Differential Revision: https://reviews.llvm.org/D67453 (detail/ViewSVN)
    by Adrian Prantl
  5. llvm-reduce: Remove inaccurate doxy comment about a return that isn't returned

    Addressing post-commit code review feedback from Dávid Bolvanský -
    thanks! (detail/ViewSVN)
    by dblaikie
  6. llvm-reduce: Fix inconsistencies between int/unsigned usage (standardize on int) (detail/ViewSVN)
    by dblaikie
  7. [analyzer] PR43102: Fix an assertion and an out-of-bounds error for diagnostic location construction

    Summary:
    https://bugs.llvm.org/show_bug.cgi?id=43102

    In today's edition of "Is this any better now that it isn't crashing?", I'd like to show you a very interesting test case with loop widening.

    Looking at the included test case, it's immediately obvious that this is not only a false positive, but also a very bad bug report in general. We can see how the analyzer mistakenly invalidated `b`, instead of its pointee, resulting in it reporting a null pointer dereference error. Not only that, the point at which this change of value is noted at is at the loop, rather then at the method call.

    It turns out that `FindLastStoreVisitor` works correctly, rather the supplied explodedgraph is faulty, because `BlockEdge` really is the `ProgramPoint` where this happens.
    {F9855739}
    So it's fair to say that this needs improving on multiple fronts. In any case, at least the crash is gone.

    Full ExplodedGraph: {F9855743}

    Reviewers: NoQ, xazax.hun, baloghadamsoftware, Charusso, dcoughlin, rnkovacs, TWeaver

    Subscribers: JesperAntonsson, uabelho, Ka-Ka, bjope, whisperity, szepet, a.sidorin, mikhail.ramalho, donat.nagy, dkrupp, gamesh411, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D66716 (detail/ViewSVN)
    by szelethus
  8. gn build: Merge r372267 (detail/ViewSVN)
    by gnsyncbot

Started by an SCM change (6 times)

This run spent:

  • 1 hr 1 min waiting;
  • 1 hr 20 min build duration;
  • 2 hr 21 min total from scheduled to completion.
LLVM/Clang Warnings: 1 warning.
    Test Result (no failures)