Started 1 mo 20 days ago
Took 18 hr on green-dragon-09

Failed Build #5460 (Sep 2, 2019 12:21:09 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 370678
  • http://llvm.org/svn/llvm-project/cfe/trunk : 370677
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 370390
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 370599
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 370527
Changes
  1. [ARM] MVE predicate bitcast test and VPSEL adjustment. NFC (detail/ViewSVN)
    by dmgreen
  2. [Wdocumentation] fixes an assertion failure with typedefed function and block pointer

    Summary:
    The assertion happens when compiling with -Wdocumentation with variable declaration to a typedefed function pointer. I not too familiar with the ObjC syntax but first two tests assert without this patch.

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

    Reviewers: gribozavr

    Reviewed By: gribozavr

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D66706 (detail/ViewSVN)
    by gribozavr
  3. [ARM] Use MQPR not QPR for MVE registers

    We should be using MQPR, and if we don't we can get COPYs and PHIs created for
    QPR. These get folded into instructions, failing verification checks.

    Differential revision: https://reviews.llvm.org/D66214 (detail/ViewSVN)
    by dmgreen
  4. [TargetLowering][PS4] Add sincos(f) lib functions when target is PS4

    PS4 supports sincosf and sincos. Adding the library functions enables
    the sin(f)+cos(f) -> sincos(f) optimization.

    Differential Revision: https://reviews.llvm.org/D67009 (detail/ViewSVN)
    by rlougher
  5. [SystemZ] Support constrained fpto[su]i intrinsics

    Now that constrained fpto[su]i intrinsic are available,
    add codegen support to the SystemZ backend.

    In addition to pure back-end changes, I've also needed
    to add the strict_fp_to_[su]int and any_fp_to_[su]int
    pattern fragments in the obvious way. (detail/ViewSVN)
    by uweigand
  6. [SVE][Inline-Asm] Support for SVE asm operands

    Summary:
    Adds the following inline asm constraints for SVE:
      - w: SVE vector register with full range, Z0 to Z31
      - x: Restricted to registers Z0 to Z15 inclusive.
      - y: Restricted to registers Z0 to Z7 inclusive.

    This change also adds the "z" modifier to interpret a register as an SVE register.

    Not all of the bitconvert patterns added by this patch are used, but they have been included here for completeness.

    Reviewers: t.p.northover, sdesmalen, rovka, momchil.velikov, rengolin, cameron.mcinally, greened

    Reviewed By: sdesmalen

    Subscribers: javed.absar, tschuett, rkruppe, psnobl, cfe-commits, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D66302 (detail/ViewSVN)
    by kmclaughlin
  7. ThinLTO: Document the option BOOTSTRAP_LLVM_ENABLE_LTO (detail/ViewSVN)
    by sylvestre
  8. [X86] getPMOVMSKB - add MVT::v64i8 handling and remove from combineBitcastvxi1. NFCI. (detail/ViewSVN)
    by rksimon
  9. Recommit r370661 "[llvm-nm] - Add a test case for case when we dump a symbol that belongs to a section with a broken sh_name."

    Fix: add a 'consumeError()' call to ObjectFile.cpp.
    This error was never checked.

    Original commit message:

    It adds a test case for a problem fixed by D66976 <https://reviews.llvm.org/D66976>.

    It was introduced by me in D66089 <https://reviews.llvm.org/D66089>.
    The error reported was never consumed because of a wrong variable name used,
    so it could fail when LLVM_ENABLE_ABI_BREAKING_CHECKS is used.

    Differential revision: https://reviews.llvm.org/D67002 (detail/ViewSVN)
    by grimar
  10. [DAGCombiner] try to form test+set out of shift+mask patterns

    The motivating bugs are:
    https://bugs.llvm.org/show_bug.cgi?id=41340
    https://bugs.llvm.org/show_bug.cgi?id=42697

    As discussed there, we could view this as a failure of IR canonicalization,
    but then we would need to implement a backend fixup with target overrides
    to get this right in all cases. Instead, we can just view this as a codegen
    opportunity. It's not even clear for x86 exactly when we should favor
    test+set; some CPUs have better theoretical throughput for the ALU ops than
    bt/test.

    This patch is made more complicated than I expected because there's an early
    DAGCombine for 'and' that can change types of the intermediate ops via
    trunc+anyext.

    Differential Revision: https://reviews.llvm.org/D66687 (detail/ViewSVN)
    by spatel
  11. Partially revert D61491 "AMDGPU: Be explicit about whether the high-word in SI_PC_ADD_REL_OFFSET is 0"

    Summary:
    D61491 caused us to use relocs when they're not strictly necessary, to
    refer to symbols in the text section. This is a pessimization and it's a
    problem for some loaders that don't support relocs yet.

    Reviewers: nhaehnle, arsenm, tpr

    Subscribers: kzhuravl, jvesely, wdng, yaxunl, dstuttard, t-tye, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D65813 (detail/ViewSVN)
    by foad
  12. [AMDGPU][MC][GFX10] Corrected constant bus checks to exclude null

    See AMD SWDEV-157286

    Reviewers: atamazov, arsenm

    Differential Revision: https://reviews.llvm.org/D65229 (detail/ViewSVN)
    by dpreobra
  13. [FileCheck] Make NumericVariable ctor explicit

    Summary:
    Make FileCheckNumericVariable constructor explicit to avoid implicit
    conversions from StringRef.

    Reviewers: jhenderson, chandlerc, jdenny, probinson, grimar, arichardson, rnk

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D66640 (detail/ViewSVN)
    by thopre
  14. [FileCheck] Forbid using var defined on same line

    Summary:
    Commit r366897 introduced the possibility to set a variable from an
    expression, such as [[#VAR2:VAR1+3]]. While introducing this feature, it
    introduced extra logic to allow using such a variable on the same line
    later on. Unfortunately that extra logic is flawed as it relies on a
    mapping from variable to expression defining it when the mapping is from
    variable definition to expression. This flaw causes among other issues
    PR42896.

    This commit avoids the problem by forbidding all use of a variable
    defined on the same line, and removes the now useless logic. Redesign
    will be done in a later commit because it will require some amount of
    refactoring first for the solution to be clean. One example is the need
    for some sort of transaction mechanism to set a variable temporarily and
    from an expression and rollback if the CHECK pattern does not match so
    that diagnostics show the right variable values.

    Reviewers: jhenderson, chandlerc, jdenny, probinson, grimar, arichardson, rnk

    Subscribers: JonChesterfield, rogfer01, hfinkel, kristina, rnk, tra, arichardson, grimar, dblaikie, probinson, llvm-commits, hiraditya

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D66141 (detail/ViewSVN)
    by thopre
  15. Revert r370661 "[llvm-nm] - Add a test case for case when we dump a symbol that belongs to a section with a broken sh_name"

    It broke BB:
    http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/16955/steps/test/logs/stdio

    Expected<T> must be checked before access or destruction.
    Unchecked Expected<T> contained error:
    a section [index 1] has an invalid sh_name (0xffff) offset which goes past the end of the section name string tableStack dump:
    0. Program arguments: /srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm /srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/test/tools/llvm-nm/Output/format-sysv-section.test.tmp2.o --format=sysv
    #0 0x00000000008af7c4 PrintStackTraceSignalHandler(void*) (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x8af7c4)
    #1 0x00000000008ad8be llvm::sys::RunSignalHandlers() (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x8ad8be)
    #2 0x00000000008afbd8 SignalHandler(int) (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x8afbd8)
    #3 0x00007f0a6b989730 __restore_rt (/lib/x86_64-linux-gnu/libpthread.so.0+0x12730)
    #4 0x00007f0a6b48d7bb raise (/lib/x86_64-linux-gnu/libc.so.6+0x377bb)
    #5 0x00007f0a6b478535 abort (/lib/x86_64-linux-gnu/libc.so.6+0x22535)
    #6 0x000000000042004b llvm::Expected<llvm::StringRef>::fatalUncheckedExpected() const (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x42004b)
    #7 0x00000000008367f5 (/sv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x8367f5)
    #8 0x0000000000817b80 llvm::object::IRObjectFile::findBitcodeInObject(llvm::object::ObjectFile const&) (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x817b80)
    #9 0x0000000000838416 llvm::object::SymbolicFile::createSymbolicFile(llvm::MemoryBufferRef, llvm::file_magic, llvm::LLVMContext*) (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x838416)
    #10 0x00000000007f36cb llvm::object::createBinary(llvm::MemoryBufferRef, llvm::LLVMContext*) (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x7f36cb)
    #11 0x0000000000413123 dumpSymbolNamesFromFile(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&) (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x413123)
    #12 0x0000000000412e38 main (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x412e38)
    #13 0x00007f0a6b47a09b __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2409b)
    #14 0x00000000004120da _start (/srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/llvm-nm+0x4120da)
    FileCheck error: '-' is empty.
    FileCheck command line:  /srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.obj/bin/FileCheck /srv/llvm-buildbot-srcatch/llvm-build-dir/clang-x86_64-debian-fast/llvm.src/test/tools/llvm-nm/format-sysv-section.test --check-prefix=ERR

    -- (detail/ViewSVN)
    by grimar
  16. [llvm-nm] - Add a test case for case when we dump a symbol that belongs to a section with a broken sh_name.

    It adds a test case for a problem fixed by D66976.

    It was introduced by me in D66089.
    The error reported was never consumed because of a wrong variable name used,
    so it could fail when LLVM_ENABLE_ABI_BREAKING_CHECKS is used.

    Differential revision: https://reviews.llvm.org/D67002 (detail/ViewSVN)
    by grimar
  17. [AMDGPU][MC][GFX10] Enabled null with 64-bit operands

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

    Reviewers: atamazov, arsenm

    https://reviews.llvm.org/D65231 (detail/ViewSVN)
    by dpreobra
  18. [InstCombine] recognize bswap disguised as shufflevector

    bitcast <N x i8> (shuf X, undef, <N, N-1,...0>) to i{N*8} --> bswap (bitcast X to i{N*8})

    In PR43146:
    https://bugs.llvm.org/show_bug.cgi?id=43146
    ...we have a more complicated case where SLP is making a mess of bswap. This patch won't
    do anything for that currently, but we need to improve bswap recognition in instcombine,
    SLP, and/or a standalone pass to avoid that problem.

    This is limited using the data-layout so we don't try to do this transform with actual
    vector types. The backend does not appear to have folds to convert in either direction,
    so we don't want to mess up something that is actually better lowered as a shuffle.

    On x86, we're trading something like this:

      vmovd %edi, %xmm0
      vpshufb LCPI0_0(%rip), %xmm0, %xmm0 ## xmm0 = xmm0[3,2,1,0,u,u,u,u,u,u,u,u,u,u,u,u]
      vmovd %xmm0, %eax

    For:

      movl %edi, %eax
      bswapl %eax

    Differential Revision: https://reviews.llvm.org/D66965 (detail/ViewSVN)
    by spatel
  19. [test] [llvm-dlltool] Improve test strictness a little. NFC. (detail/ViewSVN)
    by mstorsjo
  20. [llvm-dlltool] Handle external and internal names with differing decoration

    Also add a missed part of the test from SVN r369747.

    Differential Revision: https://reviews.llvm.org/D66996 (detail/ViewSVN)
    by mstorsjo
  21. [llvm-dlltool] Remove support for implying output name

    I don't see GNU dlltool supporting doing this; with only a -d option
    and no -l option, GNU dlltool runs successfully but doesn't write any
    output file.

    Differential Revision: https://reviews.llvm.org/D65645 (detail/ViewSVN)
    by mstorsjo
  22. [AMDGPU][MC][GFX10] Corrected constant bus limit for 64-bit shift instructions

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

    Reviewers: atamazov, arsenm

    Differential Revision: https://reviews.llvm.org/D65228 (detail/ViewSVN)
    by dpreobra
  23. [unittests][AST] CommentParser: don't name variable 'DEBUG'

    It's may be an already-defined macro name,
    resulting in compilation errors. (detail/ViewSVN)
    by lebedevri
  24. [X86][BtVer2] Fix latency and throughput of conditional SIMD store instructions.

    On BtVer2 conditional SIMD stores are heavily microcoded.
    The latency is directly proportional to the number of packed elements extracted
    from the input vector. Also, according to micro-benchmarks, most of the
    computation seems to be done in the integer unit.

    Only a minority of the uOPs is executed by the FPU. The observed behaviour on
    the FPU looks similar to this:
    - The input MASK value is moved to the Integer Unit
       -- [ a VMOVMSK-like uOP-executed on JFPU0].
    - In parallel, each element of the input XMM/YMM is extracted and then sent to
       the IntegerUnit through JFPU1.

    As expected, a (conditional) store is executed for every extracted element.
    Interestingly, a (speculative) load is executed for every extracted element too.
    It is as-if a "LOAD - BIT_EXTRACT- CMOV" sequence of uOPs is repeated by the
    integer unit for every contionally stored element.
    VMASKMOVDQU is a special case: the number of speculative loads is always 2
    (presumably, one load per quadword). That means, extra shifts and masking is
    performed on (one of) the loaded quadwords before each conditional store (that
    also explains the big number of non-FP uOPs retired).

    This patch replaces the existing writes for conditional SIMD stores (i.e.
    WriteFMaskedStore, and WriteFMaskedStoreY) with the following new writes:

      WriteFMaskedStore32  [ XMM Packed Single ]
      WriteFMaskedStore32Y [ YMM Packed Single ]
      WriteFMaskedStore64  [ XMM Packed Double ]
      WriteFMaskedStore64Y [ YMM Packed Double ]

    Added a wrapper class named X86SchedWriteMaskMove in X86Schedule.td to describe
    both RM and MR variants for conditional SIMD moves in a single tablegen
    definition.
    Instances of that class are then passed in input to multiclass avx_movmask_rm
    when constructing MASKMOVPS/PD definitions.

    Since this patch introduces new writes, I had to update all the X86 scheduling
    models.

    Differential Revision: https://reviews.llvm.org/D66801 (detail/ViewSVN)
    by adibiagio
  25. [DebugInfo] LiveDebugValues: correctly discriminate kinds of variable locations

    The missing line added by this patch ensures that only spilt variable
    locations are candidates for being restored from the stack. Otherwise,
    register or constant-value information can be interpreted as a spill
    location, through a union.

    The added regression test replicates a scenario where this occurs: the
    stack load from [rsp] causes the register-location DBG_VALUE to be
    "restored" to rsi, when it should be left alone. See PR43058 for details.

    Un x-fail a test that was suffering from this from a previous patch.

    Differential Revision: https://reviews.llvm.org/D66895 (detail/ViewSVN)
    by jmorse
  26. [llvm-strings][test] Merge two closely related tests

    This is a follow-up to feedback on D66015.

    Reviewed by: grimar

    Differential Revision: https://reviews.llvm.org/D67069 (detail/ViewSVN)
    by jhenderson
  27. Revert [Clang Interpreter] Initial patch for the constexpr interpreter

    This reverts r370636 (git commit 8327fed9475a14c3376b4860c75370c730e08f33) (detail/ViewSVN)
    by nand
  28. [AST] AST structural equivalence to work internally with pairs.

    Summary:
    The structural equivalence check stores now pairs of nodes in the
    'from' and 'to' context instead of only the node in 'from' context
    and a corresponding one in 'to' context. This is needed to handle
    cases when a Decl in the 'from' context is to be compared with
    multiple Decls in the 'to' context.

    Reviewers: martong, a_sidorin

    Reviewed By: martong, a_sidorin

    Subscribers: rnkovacs, dkrupp, Szelethus, gamesh411, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D66538 (detail/ViewSVN)
    by balazske
  29. [X86] combineHorizontalPredicateResult - pull out repeated getTargetLoweringInfo() calls. NFCI. (detail/ViewSVN)
    by rksimon
  30. [Clang Interpreter] Initial patch for the constexpr interpreter

    Summary:
    This patch introduces the skeleton of the constexpr interpreter,
    capable of evaluating a simple constexpr functions consisting of
    if statements. The interpreter is described in more detail in the
    RFC. Further patches will add more features.

    Reviewers: Bigcheese, jfb, rsmith

    Subscribers: bruno, uenoku, ldionne, Tyker, thegameg, tschuett, dexonsmith, mgorny, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64146 (detail/ViewSVN)
    by nand
  31. [AMDGPU] Add test

    Summary:
    Add test checking that the redundant immediate MOV instruction
    (by-product of handling phi nodes) is not found in the generated code.

    Reviewers: arsenm, anton-afanasyev, craig.topper, rtereshin, bogner

    Reviewed By: arsenm

    Subscribers: kzhuravl, yaxunl, dstuttard, tpr, t-tye, wdng, jvesely, nhaehnle, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D63860 (detail/ViewSVN)
    by Piotr Sobczak
  32. [yaml2obj] - Allow overriding sh_name fields of the sections.

    This is in line with the previous changes which allowed to
    override the sh_offset/sh_size and useful for writing test cases.

    Differential revision: https://reviews.llvm.org/D66998 (detail/ViewSVN)
    by grimar
  33. [DWARFVerifier] Verify GNU extensions of call site DWARF symbols

    Verify that the call site DWARF symbols (added during the implementation
    of the debug entry values feature) are generated properly.

    Differential Revision: https://reviews.llvm.org/D66865 (detail/ViewSVN)
    by djtodoro
  34. [AArch64][GlobalISel] Fix zext narrowScalar to use the right type when creating
    the merges.

    Fixes PR43171. (detail/ViewSVN)
    by aemerson
  35. [ASTImporter] At import of records re-order indirect fields too.

    Summary:
    Correct order of fields and indirect fields in imported RecordDecl
    is needed for correct work of record layout calculations.

    Reviewers: martong, a.sidorin, shafik, a_sidorin

    Reviewed By: martong, a_sidorin

    Subscribers: rnkovacs, dkrupp, Szelethus, gamesh411, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D66866 (detail/ViewSVN)
    by balazske

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

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

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

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

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

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

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

This run spent:

  • 9 hr 20 min waiting;
  • 18 hr build duration;
  • 1 day 3 hr 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

Compile Error

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