Started 7 days 12 hr ago
Took 1 hr 32 min on green-dragon-17

Failed Build rL:371447 - C:371451 - #518 (Sep 9, 2019 1:23:16 PM)

  • : 371447
  • : 371451
  • : 371446
  • : 364589
  • : 371154
  • : 371324
  • : 371422
  1. [analyzer] NFC: Simplify bug report equivalence classes to not be ilists.

    Use a vector of unique pointers instead.

    Differential Revision: (detail/ViewSVN)
    by dergachev
  2. [analyzer] NFC: Introduce sub-classes for path-sensitive and basic reports.

    Checkers are now required to specify whether they're creating a
    path-sensitive report or a path-insensitive report by constructing an
    object of the respective type.

    This makes BugReporter more independent from the rest of the Static Analyzer
    because all Analyzer-specific code is now in sub-classes.

    Differential Revision: (detail/ViewSVN)
    by dergachev
  3. Relax opcode checks in test to check for only a number instead of a specific number. (detail/ViewSVN)
    by dyung
  4. [TSan] Add AnnotateIgnoreReadsBegin declaration to tsan/test.h

    Declare the family of AnnotateIgnore[Read,Write][Begin,End] TSan
    annotations in compiler-rt/test/tsan/test.h so that we don't have to
    declare them separately in every test that needs them.  Replace usages.

    Leave usages that explicitly test the annotation mechanism:
      thread_end_with_ignore3.cpp (detail/ViewSVN)
    by yln
  5. [SDAG] Add a isSimple cover functon to MemSDNode, just as we have in IR/MI [NFC]

    Uses are in reviews D66322 and D66318.  Submitted separately to control rebuild times. (detail/ViewSVN)
    by reames
  6. [Driver] Handle default case in refactored addOpenMPRuntime

    Appease failed builds (due to -Werror and -Wswitch) where OMPRT_Unknown
    is not handled in the switch statement (even though it's handled by the
    early exit).

    This fixes -Wswitch triggered by r371442.

    Reviewers: srhines, danalbert, jdoerfert

    Subscribers: guansong, cfe-commits

    Tags: #clang

    Differential Revision: (detail/ViewSVN)
    by pirama
  7. [Remarks] Fix warning for uint8_t < 0 comparison (detail/ViewSVN)
    by thegameg
  8. [UBSan] Do not overwrite the default print_summary sanitizer option.

    This option is true by default in sanitizer common. The default
    false value was added a while ago without any reasoning in

    so, presumably it's safe to remove for consistency.

    Reviewers: hctim, samsonov, morehouse, kcc, vitalybuka

    Reviewed By: hctim, samsonov, vitalybuka

    Subscribers: delcypher, #sanitizers, llvm-commits, kcc

    Tags: #llvm, #sanitizers

    Differential Revision: (detail/ViewSVN)
    by dor1s
  9. Introduce infrastructure for an incremental port of SelectionDAG atomic load/store handling

    This is the first patch in a large sequence. The eventual goal is to have unordered atomic loads and stores - and possibly ordered atomics as well - handled through the normal ISEL codepaths for loads and stores. Today, there handled w/instances of AtomicSDNodes. The result of which is that all transforms need to be duplicated to work for unordered atomics. The benefit of the current design is that it's harder to introduce a silent miscompile by adding an transform which forgets about atomicity.  See the thread on llvm-dev titled "FYI: proposed changes to atomic load/store in SelectionDAG" for further context.

    Note that this patch is NFC unless the experimental flag is set.

    The basic strategy I plan on taking is:

        introduce infrastructure and a flag for testing (this patch)
        Audit uses of isVolatile, and apply isAtomic conservatively*
        piecemeal conservative* update generic code and x86 backedge code in individual reviews w/tests for cases which didn't check volatile, but can be found with inspection
        flip the flag at the end (with minimal diffs)
        Work through todo list identified in (2) and (3) exposing performance ops

    (*) The "conservative" bit here is aimed at minimizing the number of diffs involved in (4). Ideally, there'd be none. In practice, getting it down to something reviewable by a human is the actual goal. Note that there are (currently) no paths which produce LoadSDNode or StoreSDNode with atomic MMOs, so we don't need to worry about preserving any behaviour there.

    We've taken a very similar strategy twice before with success - once at IR level, and once at the MI level (post ISEL).

    Differential Revision: (detail/ViewSVN)
    by reames

Started by an SCM change (7 times)

This run spent:

  • 56 min waiting;
  • 1 hr 32 min build duration;
  • 2 hr 28 min total from scheduled to completion.
LLVM/Clang Warnings: 2 warnings.

Identified problems

Regression test failed

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

Compile Error

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

Ninja target failed

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

Missing test results

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