Started 8 mo 1 day ago
Took 3 min 45 sec on green-dragon-15

Failed Build #300 (Feb 17, 2019 5:23:36 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 354240
  • http://llvm.org/svn/llvm-project/cfe/trunk : 354235
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 354231
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 346271
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 354212
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 354228
Changes
  1. Revert [NFC] Better encapsulation of llvm::Optional Storage (detail/ViewSVN)
    by serge_sans_paille
  2. [NFC] Better encapsulation of llvm::Optional Storage, part II

    Fix for better Windows support. (detail/ViewSVN)
    by serge_sans_paille
  3. [NFC] Better encapsulation of llvm::Optional Storage

    Second attempt, trying to navigate out of the UB zone using
    union for storage instead of raw bytes.

    I'm prepared to revert that commit as soon as validation breaks,
    which is likely to happen. (detail/ViewSVN)
    by serge_sans_paille
  4. [LLVM-C] Add bindings to create enumerators

    Summary: The C API don't have the bindings to create enumerators, needed to create an enumeration.

    Reviewers: whitequark, CodaFi, harlanhaskins, deadalnix

    Reviewed By: whitequark, CodaFi, harlanhaskins

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D58323 (detail/ViewSVN)
    by codafi
  5. [InstCombine] add even more tests for unsigned saturated add; NFC

    The pattern-matching from rL354221 / rL354224 doesn't cover
    these, so we're up to 8 different commuted possibilities.

    There may still be 1 more variant of this pattern. (detail/ViewSVN)
    by spatel
  6. [analyzer] Make valist.Uninitialized depend on ValistBase

    Accidentally left this dependency out, resulting in an assert failure if
    only valist.Uninitialized is enabled from the valist package. (detail/ViewSVN)
    by szelethus
  7. [X86] In FP_TO_INTHelper, when moving data from SSE register to X87 register file via the stack, use the same stack slot we use for the integer conversion.

    No need for a separate stack slot. The lifetimes don't overlap.

    Also fix the MachinePointerInfo for the final load after the integer conversion to indicate it came from the stack slot. (detail/ViewSVN)
    by ctopper
  8. [NFC] Add a llvm_unreachable to silence a warning in SubstObjCTypeArgsVisitor

    All cases in the switch are covered. NFC. (detail/ViewSVN)
    by brunoricci
  9. Recommit "[AST] Factor out the logic of the various Expr::Ignore*"

    (Originally commited in r354215 and reverted in r354216 because of a
    missed failing clang-tidy test (fix in r354228))

    Now that the implementation of all of the Expr::Ignore* is in Expr.cpp
    we can try to remove some duplication. Do this by separating the logic
    of the Expr::Ignore* from the iterative loop.

    This is NFC, except for one change: IgnoreParenImpCasts now skips,
    among other things, everything that IgnoreImpCasts skips. This means
    FullExpr are now skipped by IgnoreParenImpCasts. This was likely an
    oversight when FullExpr was added to the nodes skipped by IgnoreImpCasts.

    Differential Revision: https://reviews.llvm.org/D57267

    Reviewed By: aaron.ballman (with comments from void and rnk) (detail/ViewSVN)
    by brunoricci
  10. [compiler-rt] Fix broken sanitizer bots (hopefully)

    According to the logs and local debugging there were two issues:
    1) tsan tests listed libc++.a before the source file. That's usually
       ok for shared libraries, but the linker will not add symbols from
       a static library unless needed at that time. As a result the tests
       that rely upon symbols from the library (and not only include the
       headers) had undefined references.
       To solve this I'm adding a new substitution %link_libcxx_tsan which
       expands to libc++.a if available.
    2) The target Fuzzer-x86_64-Test linked in SANITIZER_TEST_CXX_LIBRARIES
       which defaults to -lstdc++. This resulted in error messages like
         hidden symbol '_ZdlPv' is not defined locally
         hidden symbol '_Znwm' is not defined locally
       when using GNU gold (ld.bfd and lld are fine). Removing the linkage
       is fine because we build a custom libc++ for that purpose. (detail/ViewSVN)
    by hahnfeld
  11. [TEST] Remove 2>&1 from tests

    Avoid confusing CHECKS with debug dumps of sets that can be printed differently. (detail/ViewSVN)
    by mkazantsev
  12. [clang-tidy][NFCI] DanglingHandleCheck: Remove a superflous IgnoreParenImpCasts

    ExprWithCleanups is currently not skipped by IgnoreParenImpCasts, but is skipped
    by IgnoreImpCasts. In view of fixing this inconsistency in D57267, remove the
    IgnoreParenImpCasts between the ReturnStmt and the ExprWithCleanups which
    is not needed since ExprWithCleanups is always created as a direct child of
    ReturnStmt (by inspection of each ReturnStmt::Create in Sema/SemaStmt.cpp).

    NFC intended. (detail/ViewSVN)
    by brunoricci
  13. [NFC] Teach getInnermostLoopFor walk up the loop trees

    This should be NFC in current use case of this method, but it will
    help to use it for solving more compex tasks in follow-up patches. (detail/ViewSVN)
    by mkazantsev
  14. [SelectionDAG] Extract [US]MULO expansion into TL method; NFC

    In preparation for supporting vector expansion.

    Add an isPostTypeLegalization flag to makeLibCall(), because this
    expansion relies on the legalized form using MERGE_VALUES. Drop
    the corresponding variant of ExpandLibCall, which is no longer used.

    Differential Revision: https://reviews.llvm.org/D58006 (detail/ViewSVN)
    by nikic
  15. [InstCombine] reduce more unsigned saturated add with 'not' op

    We want to use the sum in the icmp to allow matching with
    m_UAddWithOverflow and eliminate the 'not'. This is discussed
    in D51929 and is another step towards solving PR14613:
    https://bugs.llvm.org/show_bug.cgi?id=14613

      Name: not op
      %notx = xor i32 %x, -1
      %a = add i32 %x, %y
      %c = icmp ult i32 %notx, %y
      %r = select i1 %c, i32 -1, i32 %a
      =>
      %a = add i32 %x, %y
      %c2 = icmp ult i32 %a, %y
      %r = select i1 %c2, i32 -1, i32 %a

      Name: not op ugt
      %notx = xor i32 %x, -1
      %a = add i32 %x, %y
      %c = icmp ugt i32 %y, %notx
      %r = select i1 %c, i32 -1, i32 %a
      =>
      %a = add i32 %x, %y
      %c2 = icmp ult i32 %a, %y
      %r = select i1 %c2, i32 -1, i32 %a

    https://rise4fun.com/Alive/niom

    (The matching here is still incomplete.) (detail/ViewSVN)
    by spatel
  16. [InstCombine] add more tests for unsigned saturated add; NFC

    Extend the pattern-matching from rL354219 / rL354221. (detail/ViewSVN)
    by spatel
  17. [RISCV] Default enable RISCV linker relaxation

    Differential Revision: https://reviews.llvm.org/D47127 (detail/ViewSVN)
    by shiva
  18. [InstCombine] reduce unsigned saturated add with 'not' op

    We want to use the sum in the icmp to allow matching with
    m_UAddWithOverflow and eliminate the 'not'. This is discussed
    in D51929 and is another step towards solving PR14613:
    https://bugs.llvm.org/show_bug.cgi?id=14613

    (The matching here is incomplete. Trying to take minimal steps
    to make sure we don't induce infinite looping from existing
    canonicalizations of the 'select'.) (detail/ViewSVN)
    by spatel

Started by upstream project Clang Stage 2: cmake, R -g Asan&UBSan, using Stage 1 RA, Phase 1 build number 6144
originally caused by:

This run spent:

  • 6.3 sec waiting;
  • 3 min 45 sec build duration;
  • 3 min 51 sec total from scheduled to completion.

Identified problems

Missing test results

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

CMake Error

This build failed because of an CMake configuration error. Below is a list of all errors in the build log:
Indication 2