Started 1 mo 3 days ago
Took 18 hr on green-dragon-08

Failed Build #5547 (Oct 18, 2019 10:30:21 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 375263
  • http://llvm.org/svn/llvm-project/cfe/trunk : 375258
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 375220
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 374854
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 375226
Changes
  1. [lit] Remove unnecessary tracking of test_index (detail/ViewSVN)
    by yln
  2. [lit] Only send back test result from worker process

    Avoid sending back the whole run.Test object (which needs to be pickled)
    from the worker process when we are only interested in the test result. (detail/ViewSVN)
    by yln
  3. [Codegen] Link MIRParser into CodeGenTests to fix MachineSizeOptsTest building (detail/ViewSVN)
    by lebedevri
  4. [NFC][CVP] Add @llvm.*.sat tests where we could prove both no-overflows (detail/ViewSVN)
    by lebedevri
  5. [Format] Add format check for throwing negative numbers

    Summary:
    The code `throw -1;` is currently formatted by clang-format as
    `throw - 1;`. This diff adds a fix for this edge case and a test to check
    for this in the future.

    For context, I am looking into a related bug in the clang-formatting of
    coroutine keywords: `co_yield -1;` is also reformatted in this manner
    as `co_yield - 1;`. A later diff will add these changes and tests for the
    `co_yield` and `co_return` keywords.

    Patch by Jonathan Thomas (jonathoma)!

    Reviewers: modocache, sammccall, Quuxplusone

    Reviewed By: sammccall

    Subscribers: cfe-commits

    Tags: #clang-format, #clang

    Differential Revision: https://reviews.llvm.org/D69144 (detail/ViewSVN)
    by modocache
  6. [DOCS]Update list of implemented constructs, NFC. (detail/ViewSVN)
    by abataev
  7. gn build: Merge r375254 (detail/ViewSVN)
    by gnsyncbot
  8. [OPENMP50]Add support for master taskloop simd.

    Added  trsing/semantics/codegen for combined construct master taskloop simd. (detail/ViewSVN)
    by abataev
  9. [PGO][PGSO] SizeOpts changes.

    Summary:
    (Split of off D67120)

    SizeOpts/MachineSizeOpts changes for profile guided size optimization.

    Reviewers: davidxl

    Subscribers: mgorny, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69070 (detail/ViewSVN)
    by yamauchi
  10. [X86] combineX86ShufflesRecursively - pull out isTargetShuffleVariableMask. NFCI. (detail/ViewSVN)
    by rksimon
  11. [IR] Reimplement FPMathOperator::classof as a whitelist.

    Summary:
    This makes it much easier to verify that the implementation matches the
    documentation. It uncovered a bug in the unit tests where we were
    accidentally setting fast math flags on a load instruction.

    Reviewers: spatel, wristow, arsenm, hfinkel, aemerson, efriedma, cameron.mcinally, mcberg2017, jmolloy

    Subscribers: wdng, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69176 (detail/ViewSVN)
    by foad
  12. Update docs for fast-math flags.

    This adds fneg, phi and select to the list of operations that may use
    fast-math flags. (detail/ViewSVN)
    by foad
  13. [clang-format] fix regression recognizing casts in Obj-C calls

    Summary:
    r373922 added checks for a few tokens that, following an `)` make it
    unlikely that the `)` is the closing paren of a cast expression. The
    specific check for `tok::l_square` there introduced a regression for
    casts of Obj-C calls, like:
    ```
    (cast)[func arg]
    ```
    From the tests added in r373922, I believe the `tok::l_square` case is added to
    capture the case where a non-cast `)` is directly followed by an
    attribute specifier, like:
    ```
    int f(int x) [[noreturn]];
    ```

    I've specialized the code to look for such attribute specifier instead
    of `tok::l_square` in general. Also, I added a regression test and moved
    the test cases added in r373922 to an already existing place documenting
    other instances of historically misidentified casts.

    Reviewers: MyDeveloperDay

    Reviewed By: MyDeveloperDay

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D69164 (detail/ViewSVN)
    by krasimir
  14. [tooling] Relax an assert when multiple GPU targets are specified. (detail/ViewSVN)
    by hliao
  15. Update MinidumpYAML to use minidump::Exception for exception stream

    Reviewers: labath, jhenderson, clayborg, MaskRay, grimar

    Reviewed By: grimar

    Subscribers: lldb-commits, grimar, MaskRay, hiraditya, llvm-commits

    Tags: #llvm, #lldb

    Differential Revision: https://reviews.llvm.org/D68657 (detail/ViewSVN)
    by josepht
  16. [AMDGPU][MC][GFX10] Added sdwa/dpp versions of v_cndmask_b32

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

    Reviewers: arsenm, rampitec

    Differential Revision: https://reviews.llvm.org/D69096 (detail/ViewSVN)
    by dpreobra
  17. [DFAPacketizer] Fix large compile-time regression for VLIW targets

    D68992 / rL375086 refactored the packetizer and removed a bunch of logic. Unfortunately it creates an Automaton object whenever a DFAPacketizer is required. These objects have no longevity, and in particular on a debug build the population of the Automaton's transition map from the underlying table is very slow (because it is called ~10 times per MachineFunction, in the testcase I'm looking at).

    This patch changes Automaton to wrap its underlying constant data in std::shared_ptr, which allows trivial copy construction. The DFAPacketizer creation function now creates a static archetypical Automaton and copies that whenever a new DFAPacketizer is required.

    This takes a testcase down from ~20s to ~0.5s in debug mode. (detail/ViewSVN)
    by jamesm
  18. Add ExceptionStream to llvm::Object::minidump

    Summary:
    This will allow updating MinidumpYAML and LLDB to use this common
    definition.

    Reviewers: labath, jhenderson, clayborg

    Reviewed By: labath

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68656 (detail/ViewSVN)
    by josepht
  19. One more attempt to fix PS4 buildbot after r375219

    PS4 buildbot seems to be dropping variable names for some reason (detail/ViewSVN)
    by evgeny777
  20. Attempt to fix PS4 buildbot after r375219 (detail/ViewSVN)
    by evgeny777
  21. Revert r375152 as it is causing failures on EXPENSIVE_CHECKS bot (detail/ViewSVN)
    by nemanjai
  22. [SCEV] Removing deprecated comment in ScalarEvolutionExpander

    Removing a comment in the ScalarEvolutionExpander.cpp file that was about the
    class SCEVSDivExpr, which has been long gone from LLVM. (detail/ViewSVN)
    by vhscampos
  23. [AMDGPU][MC][GFX9] Corrected parsing of v_cndmask_b32_sdwa

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

    Reviewers: arsenm, rampitec

    Differential Revision: https://reviews.llvm.org/D69095 (detail/ViewSVN)
    by dpreobra
  24. [NFC][CVP] Count all the no-wraps we proved

    Summary:
    It looks like this is the only missing statistic in the CVP pass.
    Since we prove NSW and NUW separately i'd think we should count them separately too.

    Reviewers: nikic, spatel, reames

    Reviewed By: spatel

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68740 (detail/ViewSVN)
    by lebedevri
  25. [AArch64] Adding support for PMMIR_EL1 register

    Summary:
    The PMMIR_EL1 register is present in Armv8.4 with PMU extension.
    This patch adds support for it.

    Reviewers: t.p.northover, dnsampaio

    Reviewed By: dnsampaio

    Subscribers: kristof.beyls, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68940 (detail/ViewSVN)
    by vhscampos
  26. [clangd] Report declaration references in findExplicitReferences.

    Reviewers: ilya-biryukov

    Subscribers: MaskRay, jkorous, arphaman, kadircet, usaxena95, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D68977 (detail/ViewSVN)
    by hokein
  27. [ThinLTOCodeGenerator] Add support for index-based WPD

    This is clang part of the patch. It adds -flto-unit flag for thin LTO
    builds on Mac and PS4

    Differential revision: https://reviews.llvm.org/D68950 (detail/ViewSVN)
    by evgeny777
  28. [AArch64][SVE] Add SPLAT_VECTOR ISD Node

    Adds a new ISD node to replicate a scalar value across all elements of
    a vector. This is needed for scalable vectors, since BUILD_VECTOR cannot
    be used.

    Fixes up default type legalization for scalable vectors after the
    new MVT type ranges were introduced.

    At present I only use this node for scalable vectors. A DAGCombine has
    been added to transform a BUILD_VECTOR into a SPLAT_VECTOR if all
    elements are the same, but only if the default operation action of
    Expand has been overridden by the target.

    I've only added result promotion legalization for scalable vector
    i8/i16/i32/i64 types in AArch64 for now.

    Reviewers: t.p.northover, javed.absar, greened, cameron.mcinally, jmolloy

    Reviewed By: jmolloy

    Differential Revision: https://reviews.llvm.org/D47775 (detail/ViewSVN)
    by huntergr
  29. [Arm][libsanitizer] Fix arm libsanitizer failure with bleeding edge glibc

    Glibc has recently introduced changed to the mode field in ipc_perm in commit
    2f959dfe849e0646e27403f2e4091536496ac0f0. For Arm this means that the mode
    field no longer has the same size.

    This causes an assert failure against libsanitizer's internal copy of ipc_perm.
    Since this change can't be easily detected I am adding arm to the list of
    targets that are excluded from this check.

    Patch by: Tamar Christina

    Differential Revision: https://reviews.llvm.org/D69104 (detail/ViewSVN)
    by sjoerdmeijer
  30. [ThinLTOCodeGenerator] Add support for index-based WPD

    Differential revision: https://reviews.llvm.org/D68950 (detail/ViewSVN)
    by evgeny777
  31. [AArch64] Don't combine callee-save and local stack adjustment when optimizing for size

    For arm64, D18619 introduced the ability to combine bumping the stack pointer
    upfront in case it needs to be bumped for both the callee-save area as well as
    the local stack area.

    That diff already remarks that "This change can cause an increase in
    instructions", but argues that even when that happens, it should be still be a
    performance benefit because the number of micro-ops is reduced.

    We have observed that this code-size increase can be significant in practice.
    This diff disables combining stack bumping for methods that are marked as
    optimize-for-size.

    Example of a prologue with the behavior before this diff (combining stack bumping when possible):
      sub        sp, sp, #0x40
      stp        d9, d8, [sp, #0x10]
      stp        x20, x19, [sp, #0x20]
      stp        x29, x30, [sp, #0x30]
      add        x29, sp, #0x30
      [... compute x8 somehow ...]
      stp        x0, x8, [sp]

    And after this  diff, if the method is marked as optimize-for-size:
      stp        d9, d8, [sp, #-0x30]!
      stp        x20, x19, [sp, #0x10]
      stp        x29, x30, [sp, #0x20]
      add        x29, sp, #0x20
      [... compute x8 somehow ...]
      stp        x0, x8, [sp, #-0x10]!

    Note that without combining the stack bump there are two auto-decrements,
    nicely folded into the stp instructions, whereas otherwise there is a single
    sub sp, ... instruction, but not folded.

    Patch by Nikolai Tillmann!

    Differential Revision: https://reviews.llvm.org/D68530 (detail/ViewSVN)
    by dmgreen
  32. [X86] Regenerate memcmp tests and add X64-AVX512 common prefix

    Should help make the changes in D69157 clearer (detail/ViewSVN)
    by rksimon
  33. Fix MSVC "not all control paths return a value" warning. NFCI. (detail/ViewSVN)
    by rksimon
  34. Fix MSVC "result of 32-bit shift implicitly converted to 64 bits" warnings. NFCI. (detail/ViewSVN)
    by rksimon
  35. [Codegen] Alter the default promotion for saturating adds and subs

    The default promotion for the add_sat/sub_sat nodes currently does:
        ANY_EXTEND iN to iM
        SHL by M-N
        [US][ADD|SUB]SAT
        L/ASHR by M-N

    If the promoted add_sat or sub_sat node is not legal, this can produce code
    that effectively does a lot of shifting (and requiring large constants to be
    materialised) just to use the overflow flag. It is simpler to just do the
    saturation manually, using the higher bitwidth addition and a min/max against
    the saturating bounds. That is what this patch attempts to do.

    Differential Revision: https://reviews.llvm.org/D68926 (detail/ViewSVN)
    by dmgreen
  36. [AArch64][SVE] Implement unpack intrinsics

    Summary:
    Implements the following intrinsics:
      - int_aarch64_sve_sunpkhi
      - int_aarch64_sve_sunpklo
      - int_aarch64_sve_uunpkhi
      - int_aarch64_sve_uunpklo

    This patch also adds AArch64ISD nodes for UNPK instead of implementing
    the intrinsics directly, as they are required for a future patch which
    implements the sign/zero extension of legal vectors.

    This patch includes tests for the Subdivide2Argument type added by D67549

    Reviewers: sdesmalen, SjoerdMeijer, greened, rengolin, rovka

    Reviewed By: greened

    Subscribers: tschuett, kristof.beyls, rkruppe, psnobl, cfe-commits, llvm-commits

    Differential Revision: https://reviews.llvm.org/D67550 (detail/ViewSVN)
    by kmclaughlin
  37. [InstCombine] Fix miscompile bug in canEvaluateShuffled

    Summary:
    Add restrictions in canEvaluateShuffled to prevent that we for example
    transform

      %0 = insertelement <2 x i16> undef, i16 %a, i32 0
      %1 = srem <2 x i16> %0, <i16 2, i16 1>
      %2 = shufflevector <2 x i16> %1, <2 x i16> undef, <2 x i32> <i32 undef, i32 0>

    into

       %1 = insertelement <2 x i16> undef, i16 %a, i32 1
       %2 = srem <2 x i16> %1, <i16 undef, i16 2>

    as having an undef denominator makes the srem undefined (for all
    vector elements).

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

    Reviewers: spatel, lebedev.ri

    Reviewed By: spatel, lebedev.ri

    Subscribers: lebedev.ri, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69038 (detail/ViewSVN)
    by bjope
  38. [InstCombine] Pre-commit of test case showing miscompile bug in canEvaluateShuffled

    Adding the reproducer from  https://bugs.llvm.org/show_bug.cgi?id=43689,
    showing that instcombine is doing a bad transform. It transforms

      %0 = insertelement <2 x i16> undef, i16 %a, i32 0
      %1 = srem <2 x i16> %0, <i16 2, i16 1>
      %2 = shufflevector <2 x i16> %1, <2 x i16> undef, <2 x i32> <i32 undef, i32 0>

    into

       %1 = insertelement <2 x i16> undef, i16 %a, i32 1
       %2 = srem <2 x i16> %1, <i16 undef, i16 2>

    The undef denominator makes the whole srem undefined. (detail/ViewSVN)
    by bjope
  39. [WebAssembly] -pthread implies -target-feature +sign-ext

    Summary:
    The sign extension proposal was motivated by a desire to not have
    separate sign-extending atomic operations, so it is meant to be
    enabled when threads are used.

    Reviewers: aheejin, dschuff

    Subscribers: sbc100, jgravelle-google, sunfish, jfb, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D69075 (detail/ViewSVN)
    by tlively
  40. [X86] Emit KTEST when possible

    https://reviews.llvm.org/D69111 (detail/ViewSVN)
    by davezarzycki
  41. [lit] Move resolving of XFAIL result codes out of Test.setResult

    This will allow us to serialize just the result object instead of the
    whole lit.Test object back from the worker to the main lit process. (detail/ViewSVN)
    by yln
  42. [lit] worker.py: Improve code for executing a single test (detail/ViewSVN)
    by yln
  43. [IndVars] Factor out some common code into a utility function

    As requested in review of D69009 (detail/ViewSVN)
    by reames
  44. [Test] Precommit test for D69006 (detail/ViewSVN)
    by reames
  45. [analyzer] exploded-graph-rewriter: Fix typo in r375186. Unbreaks tests. (detail/ViewSVN)
    by dergachev
  46. [analyzer] Assign truly stable identifiers to exploded nodes.

    ExplodedGraph nodes will now have a numeric identifier stored in them
    which will keep track of the order in which the nodes were created
    and it will be fully deterministic both accross runs and across machines.

    This is extremely useful for debugging as it allows reliably setting
    conditional breakpoints by node IDs. (detail/ViewSVN)
    by dergachev
  47. [analyzer] Display cast kinds in program point dumps.

    Because cast expressions have their own hierarchy, it's extremely useful
    to have some information about what kind of casts are we dealing with. (detail/ViewSVN)
    by dergachev
  48. [analyzer] exploded-graph-rewriter: Make node headers a bit lighter.

    The 50% grey color is too dark on some monitors. (detail/ViewSVN)
    by dergachev
  49. DebugInfo: Move loclist base address from DwarfFile to DebugLocStream

    There's no need to have more than one of these (there can be two
    DwarfFiles - one for the .o, one for the .dwo - but only one loc/loclist
    section (either in the .o or the .dwo) & certainly one per
    DebugLocStream, which is currently singular in DwarfDebug) (detail/ViewSVN)
    by dblaikie

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18551
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18553
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18554
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18555
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18556
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18557
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18558
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18559
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18560
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18561
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18562
originally caused by:

This run spent:

  • 17 hr waiting;
  • 18 hr build duration;
  • 1 day 12 hr total from scheduled to completion.

Identified problems

Compile Error

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

Regression test failed

This build failed because a regression test in the test suite FAILed. See the test report for details.
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