Started 1 mo 0 days ago
Took 5 hr 6 min on green-dragon-02

Failed Build #14900 (Oct 17, 2019 2:42:48 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 375090
  • http://llvm.org/svn/llvm-project/cfe/trunk : 375079
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 374977
  • http://llvm.org/svn/llvm-project/zorg/trunk : 375080
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 374854
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 375058
Changes
  1. Try to fix the assert in Alignment::alignAddr to work on 32-bit

    Hopefully fixing the AlignmentDeathTest.AlignAddr failures (e.g. at
    http://lab.llvm.org:8011/builders/clang-cmake-armv7-quick/builds/10925) (detail)
    by hans
  2. [Analysis] Don't assume that unsigned overflow can't happen in EmitGEPOffset (PR42699)

    Summary:
    Currently when computing a GEP offset using the function EmitGEPOffset
    for the following instruction

      getelementptr inbounds i32, i32* %p, i64 %offs

    we get

      mul nuw i64 %offs, 4

    Unfortunately we cannot assume that unsigned wrapping won't happen
    here because %offs is allowed to be negative.

    Making such assumptions can lead to miscompilations: see the new test
    test24_neg_offs in InstCombine/icmp.ll. Without the patch InstCombine
    would generate the following comparison:

       icmp eq i64 %offs, 4611686018427387902; 0x3ffffffffffffffe

    Whereas the correct value to compare with is -2.

    This patch replaces the NUW flag with NSW in the multiplication
    instructions generated by EmitGEPOffset and adjusts the test suite.

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

    Reviewers: chandlerc, craig.topper, ostannard, lebedev.ri, spatel, efriedma, nlopes, aqjune

    Reviewed By: lebedev.ri

    Subscribers: reames, lebedev.ri, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68342 (detail)
    by miyuki
  3. Revert r374931 "[llvm-objdump] Use a counter for llvm-objdump -h instead of the section index."

    This broke llvm-objdump in 32-bit builds, see e.g.
    http://lab.llvm.org:8011/builders/clang-cmake-armv7-quick/builds/10925

    > Summary:
    > When listing the index in `llvm-objdump -h`, use a zero-based counter instead of the actual section index (e.g. shdr->sh_index for ELF).
    >
    > While this is effectively a noop for now (except one unit test for XCOFF), the index values will change in a future patch that filters certain sections out (e.g. symbol tables). See D68669 for more context. Note: the test case in `test/tools/llvm-objdump/X86/section-index.s` already covers the case of incrementing the section index counter when sections are skipped.
    >
    > Reviewers: grimar, jhenderson, espindola
    >
    > Reviewed By: grimar
    >
    > Subscribers: emaste, sbc100, arichardson, aheejin, arphaman, seiya, llvm-commits, MaskRay
    >
    > Tags: #llvm
    >
    > Differential Revision: https://reviews.llvm.org/D68848 (detail)
    by hans
  4. [ARM][MVE] Change VPST to use, not def, VPR

    Unlike VPT, VPST just uses the current value of VPR.P0.

    Differential Revision: https://reviews.llvm.org/D69037 (detail)
    by sam_parker
  5. [DFAPacketizer] Use DFAEmitter. NFC.

    Summary:
    This is a NFC change that removes the NFA->DFA construction and emission logic from DFAPacketizerEmitter and instead uses the generic DFAEmitter logic. This allows DFAPacketizer to use the Automaton class from Support and remove a bunch of logic there too.

    After this patch, DFAPacketizer is mostly logic for grepping Itineraries and collecting functional units, with no state machine logic. This will allow us to modernize by removing the 16-functional-unit limit and supporting non-itinerary functional units. This is all for followup patches.

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68992 (detail)
    by jamesm
  6. [DAGCombine][ARM] Enable extending masked loads

    Add generic DAG combine for extending masked loads.

    Allow us to generate sext/zext masked loads which can access v4i8,
    v8i8 and v4i16 memory to produce v4i32, v8i16 and v4i32 respectively.

    Differential Revision: https://reviews.llvm.org/D68337 (detail)
    by sam_parker
  7. [Alignment][NFC] Use Align for TargetFrameLowering/Subtarget

    Summary:
    This is patch is part of a series to introduce an Alignment type.
    See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html
    See this patch for the introduction of the type: https://reviews.llvm.org/D64790

    Reviewers: courbet

    Subscribers: jholewinski, arsenm, dschuff, jyknight, dylanmckay, sdardis, nemanjai, jvesely, nhaehnle, sbc100, jgravelle-google, hiraditya, aheejin, kbarton, fedor.sergeev, asb, rbar, johnrusso, simoncook, apazos, sabuasal, niosHD, jrtc27, MaskRay, zzheng, edward-jones, atanasyan, rogfer01, MartinMosbeck, brucehoult, the_o, PkmX, jocewei, jsji, Jim, lenary, s.egerton, pzheng, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68993 (detail)
    by gchatelet
  8. [ThinLTO] Import virtual method with single implementation in hybrid mode

    Differential revision: https://reviews.llvm.org/D68782 (detail)
    by evgeny777
  9. [llvm-ar] Simplify and make two global variables static. NFC (detail)
    by maskray

Started by timer (4 times)

This run spent:

  • 3 hr 31 min waiting;
  • 5 hr 6 min build duration;
  • 8 hr 37 min total from scheduled to completion.

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

Ninja target failed

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

Missing test results

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