Started 2 mo 24 days ago
Took 1 hr 28 min on green-dragon-20

Success Build rL:372256 - C:372252 - #662 (Sep 18, 2019 12:55:13 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 372256
  • http://llvm.org/svn/llvm-project/cfe/trunk : 372252
  • 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. [AArch64] Don't implicitly enable global isel on Darwin if code-model==large.

    Summary:
    AArch64 GlobalISel doesn't support MachO's large code model, so this patch
    adds a check for that combination before implicitly enabling it.

    Reviewers: paquette

    Subscribers: kristof.beyls, ributzka, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D67724 (detail/ViewSVN)
    by Lang Hames
  2. [SimplifyCFG] mergeConditionalStoreToAddress(): consider cost, not instruction count

    Summary:
    As it can be see in the changed test, while `div` is really costly,
    we were speculating it. This does not seem correct.

    Also, the old code would run for every single insturuction in BB,
    instead of eagerly bailing out as soon as there are too many instructions.

    This function still has a problem that `PHINodeFoldingThreshold` is
    per-basic-block, while it should be for all the basic blocks.

    Reviewers: efriedma, craig.topper, dmgreen, jmolloy

    Reviewed By: jmolloy

    Subscribers: xbolva00, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D67315 (detail/ViewSVN)
    by lebedevri
  3. [MIPS] For vectors, select `add %x, C` as `sub %x, -C` if it results in inline immediate

    Summary:
    As discussed in https://reviews.llvm.org/D62341#1515637,
    for MIPS `add %x, -1` isn't optimal. Unlike X86 there
    are no fastpaths to matearialize such `-1`/`1` vector constants,
    and `sub %x, 1` results in better codegen,
    so undo canonicalization

    Reviewers: atanasyan, Petar.Avramovic, RKSimon

    Reviewed By: atanasyan

    Subscribers: sdardis, arichardson, hiraditya, jrtc27, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D66805 (detail/ViewSVN)
    by lebedevri
  4. [CodeGen][MIPS][NFC] Some standalone tests for D66805 "or vectors, select `add %x, C` as `sub %x, -C` if it results in inline immediate" (detail/ViewSVN)
    by lebedevri
  5. [OPENMP]Fix for PR43349: Crash for privatized loop bound.

    If the variable, used in the loop boundaries, is not captured in the
    construct, this variable must be considered as undefined if it was
    privatized. (detail/ViewSVN)
    by abataev
  6. [mips] Expand 'lw/sw' instructions for 32-bit GOT

    In case of using 32-bit GOT access to the table requires two instructions
    with attached %got_hi and %got_lo relocations. This patch implements
    correct expansion of 'lw/sw' instructions in that case.

    Differential Revision: https://reviews.llvm.org/D67705 (detail/ViewSVN)
    by atanasyan
  7. actually also compile output in tests for -frewrite-includes

    Checking that the created output matches something is nice, but
    this should also check whether the output makes sense.

    Differential Revision: https://reviews.llvm.org/D63979 (detail/ViewSVN)
    by llunak
  8. [clang-format][PR41899] PointerAlignment: Left leads to useless space in lambda intializer expression

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

    ```auto lambda = [&a = a]() { a = 2; };```

    is formatted as

    ```auto lambda = [& a = a]() { a = 2; };```

    With an extra space if PointerAlignment is set to Left

    > The space "& a" looks strange when there is no type in the lambda's intializer expression. This can be worked around with by setting "PointerAlignment: Right", but ideally "PointerAlignment: Left" would not add a space in this case.

    Reviewers: klimek, owenpan, krasimir, timwoj

    Reviewed By: klimek

    Subscribers: cfe-commits

    Tags: #clang-tools-extra, #clang

    Differential Revision: https://reviews.llvm.org/D67718 (detail/ViewSVN)
    by paulhoad
  9. make -frewrite-includes also rewrite conditions in #if/#elif

    Those conditions may use __has_include, which needs to be rewritten.
    The existing code has already tried to rewrite just __has_include,
    but it didn't work with macro expansion, so e.g. Qt's
    "#define QT_HAS_INCLUDE(x) __has_include(x)" didn't get handled
    properly. Since the preprocessor run knows what each condition evaluates
    to, just rewrite the entire condition. This of course requires that
    the -frewrite-include pass has the same setup as the following
    compilation, but that has always been the requirement.

    Differential Revision: https://reviews.llvm.org/D63508 (detail/ViewSVN)
    by llunak

Started by an SCM change (8 times)

This run spent:

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