Started 1 mo 4 days ago
Took 4 hr 18 min on green-dragon-02

Failed Build #14850 (Oct 8, 2019 4:04:31 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 374128
  • http://llvm.org/svn/llvm-project/cfe/trunk : 374127
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 374125
  • http://llvm.org/svn/llvm-project/zorg/trunk : 374004
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 374120
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 374103
Changes
  1. [Docs] Fixes broken sphinx build - undefined label

    Removes label ref pointing to non-existent subsystem docs page. (detail)
    by dr87
  2. [clang-scan-deps] Improve string/character literal skipping

    The existing string/character literal skipping code in the
    dependency directives source minimizer has two issues:

    - It doesn't stop the scanning when a newline is reached before the terminating character,
    unlike the lexer which considers the token to be done (even if it's invalid) at the end of the line.

    - It doesn't support whitespace between '\' and the newline when looking if the '\' is used as a line continuation character.

    This commit fixes both issues.

    Differential Revision: https://reviews.llvm.org/D68436 (detail)
    by arphaman
  3. [IRGen] Emit lifetime markers for temporary struct allocas

    When passing arguments using temporary allocas, we need to add the
    appropriate lifetime markers so that the stack coloring passes can
    re-use the stack space.

    This patch keeps track of all the lifetime.start calls emited before the
    codegened call, and adds the corresponding lifetime.end calls after the
    call.

    Differential Revision: https://reviews.llvm.org/D68611 (detail)
    by thegameg
  4. [sanitizer] Fix crypt.cpp on Android again (detail)
    by Vitaly Buka
  5. [IA] Add tests for a few other edge cases

    Test with the last eight bits within the range [7F, FF] and with
    lower-case hex letters. (detail)
    by void
  6. [dsymutil] Improve verbose output (NFC)

    The verbose output for finding relocations assumed that we'd always dump
    the DIE after (which starts with a newline) and therefore didn't include
    one itself. However, this isn't always true, leading to garbled output.

    This patch adds a newline to the verbose output and adds a line that
    says that the DIE is being kept (which isn't obvious otherwise). It also
    adds a 0x prefix to the relocations. (detail)
    by Jonas Devlieghere
  7. DebugInfo: Move LLE enum handling to .def to match RLE handling (detail)
    by dblaikie
  8. [libc++] Workaround old versions of CMake that don't understand list(JOIN) (detail)
    by Louis Dionne
  9. Fix crash or wrong code bug if a lifetime-extended temporary contains a
    "non-constant" value.

    If the constant evaluator evaluates part of a variable initializer,
    including the initializer for some lifetime-extended temporary, but
    fails to fully evaluate the initializer, it can leave behind wrong
    values for temporaries encountered in that initialization. Don't try to
    emit those from CodeGen! Instead, look at the values that constant
    evaluation produced if (and only if) it actually succeeds and we're
    emitting the lifetime-extending declaration's initializer as a constant. (detail)
    by rsmith
  10. [libc++] Move the linker script generation step to CMake

    Summary:
    This allows the linker script generation to query CMake properties
    (specifically the dependencies of libc++.so) instead of having to
    carry these dependencies around manually in global variables. Notice
    the removal of the LIBCXX_INTERFACE_LIBRARIES global variable.

    Reviewers: phosek, EricWF

    Subscribers: mgorny, christof, jkorous, dexonsmith, libcxx-commits

    Tags: #libc

    Differential Revision: https://reviews.llvm.org/D68343 (detail)
    by Louis Dionne
  11. [sanitizer] Fix crypt.cpp test on Darwin (detail)
    by Vitaly Buka
  12. [clang] enable_trivial_var_init_zero should not be Joined<>

    Reviewers: rnk

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D68610 (detail)
    by Vitaly Buka
  13. [CVP} Replace SExt with ZExt if the input is known-non-negative

    Summary:
    zero-extension is far more friendly for further analysis.
    While this doesn't directly help with the shift-by-signext problem, this is not unrelated.

    This has the following effect on test-suite (numbers collected after the finish of middle-end module pass manager):
    | Statistic                            |     old |     new | delta | percent change |
    | correlated-value-propagation.NumSExt |       0 |    6026 |  6026 |   +100.00%     |
    | instcount.NumAddInst                 |  272860 |  271283 | -1577 |     -0.58%     |
    | instcount.NumAllocaInst              |   27227 |   27226 | -1    |      0.00%     |
    | instcount.NumAndInst                 |   63502 |   63320 | -182  |     -0.29%     |
    | instcount.NumAShrInst                |   13498 |   13407 | -91   |     -0.67%     |
    | instcount.NumAtomicCmpXchgInst       |    1159 |    1159 |  0    |      0.00%     |
    | instcount.NumAtomicRMWInst           |    5036 |    5036 |  0    |      0.00%     |
    | instcount.NumBitCastInst             |  672482 |  672353 | -129  |     -0.02%     |
    | instcount.NumBrInst                  |  702768 |  702195 | -573  |     -0.08%     |
    | instcount.NumCallInst                |  518285 |  518205 | -80   |     -0.02%     |
    | instcount.NumExtractElementInst      |   18481 |   18482 |  1    |      0.01%     |
    | instcount.NumExtractValueInst        |   18290 |   18288 | -2    |     -0.01%     |
    | instcount.NumFAddInst                |  139035 |  138963 | -72   |     -0.05%     |
    | instcount.NumFCmpInst                |   10358 |   10348 | -10   |     -0.10%     |
    | instcount.NumFDivInst                |   30310 |   30302 | -8    |     -0.03%     |
    | instcount.NumFenceInst               |     387 |     387 |  0    |      0.00%     |
    | instcount.NumFMulInst                |   93873 |   93806 | -67   |     -0.07%     |
    | instcount.NumFPExtInst               |    7148 |    7144 | -4    |     -0.06%     |
    | instcount.NumFPToSIInst              |    2823 |    2838 |  15   |      0.53%     |
    | instcount.NumFPToUIInst              |    1251 |    1251 |  0    |      0.00%     |
    | instcount.NumFPTruncInst             |    2195 |    2191 | -4    |     -0.18%     |
    | instcount.NumFSubInst                |   92109 |   92103 | -6    |     -0.01%     |
    | instcount.NumGetElementPtrInst       | 1221423 | 1219157 | -2266 |     -0.19%     |
    | instcount.NumICmpInst                |  479140 |  478929 | -211  |     -0.04%     |
    | instcount.NumIndirectBrInst          |       2 |       2 |  0    |      0.00%     |
    | instcount.NumInsertElementInst       |   66089 |   66094 |  5    |      0.01%     |
    | instcount.NumInsertValueInst         |    2032 |    2030 | -2    |     -0.10%     |
    | instcount.NumIntToPtrInst            |   19641 |   19641 |  0    |      0.00%     |
    | instcount.NumInvokeInst              |   21789 |   21788 | -1    |      0.00%     |
    | instcount.NumLandingPadInst          |   12051 |   12051 |  0    |      0.00%     |
    | instcount.NumLoadInst                |  880079 |  878673 | -1406 |     -0.16%     |
    | instcount.NumLShrInst                |   25919 |   25921 |  2    |      0.01%     |
    | instcount.NumMulInst                 |   42416 |   42417 |  1    |      0.00%     |
    | instcount.NumOrInst                  |  100826 |  100576 | -250  |     -0.25%     |
    | instcount.NumPHIInst                 |  315118 |  314092 | -1026 |     -0.33%     |
    | instcount.NumPtrToIntInst            |   15933 |   15939 |  6    |      0.04%     |
    | instcount.NumResumeInst              |    2156 |    2156 |  0    |      0.00%     |
    | instcount.NumRetInst                 |   84485 |   84484 | -1    |      0.00%     |
    | instcount.NumSDivInst                |    8599 |    8597 | -2    |     -0.02%     |
    | instcount.NumSelectInst              |   45577 |   45913 |  336  |      0.74%     |
    | instcount.NumSExtInst                |   84026 |   78278 | -5748 |     -6.84%     |
    | instcount.NumShlInst                 |   39796 |   39726 | -70   |     -0.18%     |
    | instcount.NumShuffleVectorInst       |  100272 |  100292 |  20   |      0.02%     |
    | instcount.NumSIToFPInst              |   29131 |   29113 | -18   |     -0.06%     |
    | instcount.NumSRemInst                |    1543 |    1543 |  0    |      0.00%     |
    | instcount.NumStoreInst               |  805394 |  804351 | -1043 |     -0.13%     |
    | instcount.NumSubInst                 |   61337 |   61414 |  77   |      0.13%     |
    | instcount.NumSwitchInst              |    8527 |    8524 | -3    |     -0.04%     |
    | instcount.NumTruncInst               |   60523 |   60484 | -39   |     -0.06%     |
    | instcount.NumUDivInst                |    2381 |    2381 |  0    |      0.00%     |
    | instcount.NumUIToFPInst              |    5549 |    5549 |  0    |      0.00%     |
    | instcount.NumUnreachableInst         |    9855 |    9855 |  0    |      0.00%     |
    | instcount.NumURemInst                |    1305 |    1305 |  0    |      0.00%     |
    | instcount.NumXorInst                 |   10230 |   10081 | -149  |     -1.46%     |
    | instcount.NumZExtInst                |   60353 |   66840 |  6487 |     10.75%     |
    | instcount.TotalBlocks                |  829582 |  829004 | -578  |     -0.07%     |
    | instcount.TotalFuncs                 |   83818 |   83817 | -1    |      0.00%     |
    | instcount.TotalInsts                 | 7316574 | 7308483 | -8091 |     -0.11%     |

    TLDR: we produce -0.11% less instructions, -6.84% less `sext`, +10.75% more `zext`.
    To be noted, clearly, not all new `zext`'s are produced by this fold.

    (And now i guess it might have been interesting to measure this for D68103 :S)

    Reviewers: nikic, spatel, reames, dberlin

    Reviewed By: nikic

    Subscribers: hiraditya, jfb, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68654 (detail)
    by lebedevri
  14. [CVP][NFC] Revisit sext vs. zext test (detail)
    by lebedevri
  15. [clang] Add llvm-ifs in test deps (detail)
    by Vitaly Buka
  16. Fix `compiler_rt_logbf_test.c` test failure for Builtins-i386-darwin test suite.

    Summary:
    It seems that compiler-rt's implementation and Darwin
    libm's implementation of `logbf()` differ when given a NaN
    with raised sign bit. Strangely this behaviour only happens with
    i386 Darwin libm. For x86_64 and x86_64h the existing compiler-rt
    implementation matched Darwin libm.

    To workaround this the `compiler_rt_logbf_test.c` has been modified
    to do a comparison on the `fp_t` type and if that fails check if both
    values are NaN. If both values are NaN they are equivalent and no
    error needs to be raised.

    rdar://problem/55565503

    Reviewers: rupprecht, scanon, compnerd, echristo
    Subscribers: #sanitizers, llvm-commits
    Tags: #llvm, #sanitizers
    Differential Revision: https://reviews.llvm.org/D67999 (detail)
    by delcypher
  17. [OPENMP50]Multiple vendors in vendor context must be treated as logical
    and of vendors, not or.

    If several vendors are provided in the same vendor context trait, the
    context shall match only if all vendors are matching, not one of them.
    This is per OpenMP 5.0, 2.3.3 Matching and Scoring Context Selectors,
    all selectors in the construct, device, and implementation sets of the
    context selector appear in the corresponding trait set of the OpenMP
    context. (detail)
    by abataev
  18. Try to get ubsan-blacklist-vfs.c pass more on Windows (detail)
    by nico
  19. Try to get readability-deleted-default.cpp to pass on Windows.

    In MS compatibility mode, "extern inline void g()" is not a redundant
    declaration for "inline void g()", because of redeclForcesDefMSVC()
    (see PR19264, r205485).

    To fix, run the test with -fms-compatiblity forced on and off
    and explicit check for the differing behavior for extern inline.

    Final bit of PR43593.

    Differential Revision: https://reviews.llvm.org/D68640 (detail)
    by nico
  20. Mark several PointerIntPair methods as lvalue-only

    No point in mutating 'this' if it's just going to be thrown away.

    https://reviews.llvm.org/D63945 (detail)
    by jrose

Started by timer (4 times)

This run spent:

  • 3 hr 53 min waiting;
  • 4 hr 18 min build duration;
  • 8 hr 12 min total from scheduled to completion.

Identified problems

Ninja target failed

Below is a link to the first failed ninja target.
Indication 1

Missing test results

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

Regression test failed

This build failed because a regression test in the test suite FAILed. See the test report for details.
Indication 3