Started 2 mo 13 days ago
Took 10 hr on green-dragon-15

Failed Build #6459 (Jul 10, 2019 7:56:09 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 365633
  • http://llvm.org/svn/llvm-project/cfe/trunk : 365629
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 365591
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 365562
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 365632
Changes
  1. gn build: Merge r365585 (detail)
    by nico
  2. [clangd] Trim spaces around parsed include in include extractor (detail)
    by kadircet
  3. [clangd] Add a flag to clangdServer rename function to control whether we want format the replacements.

    Summary:
    This would allow clangd embedders to use the ClangdServer::rename for other
    purposes (highlighting all the occurrences of the symbol in prepare
    stage).

    Reviewers: sammccall, ilya-biryukov

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

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64481 (detail)
    by hokein
  4. [docs][llvm-symbolizer] Fix grammar (detail)
    by jhenderson
  5. Loop pragma parsing. NFC.

    I would like to add some pragma handling here, but couldn't resist a little NFC
    and tidy up first.

    Differential Revision: https://reviews.llvm.org/D64471 (detail)
    by sjoerdmeijer
  6. [X86] EltsFromConsecutiveLoads - cleanup Zero/Undef/Load element collection. NFCI. (detail)
    by rksimon
  7. [MIPS GlobalISel] Select float and double phi

    Select float and double phi for MIPS32.

    Differential Revision: https://reviews.llvm.org/D64420 (detail)
    by petar.avramovic
  8. [MIPS GlobalISel] Select float and double load and store

    Select float and double load and store for MIPS32.

    Differential Revision: https://reviews.llvm.org/D64419 (detail)
    by petar.avramovic
  9. [FileCheck] Simplify numeric variable interface

    Summary:
    This patch simplifies 2 aspects in the FileCheckNumericVariable code.

    First, setValue() method is turned into a void function since being
    called only on undefined variable is an invariant and is now asserted
    rather than returned. This remove the assert from the callers.

    Second, clearValue() method is also turned into a void function since
    the only caller does not check its return value since it may be trying
    to clear the value of variable that is already cleared without this
    being noteworthy.

    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/D64231

    llvm-svn: 365249 (detail)
    by thopre
  10. [FileCheck] Fix @LINE value after match failure

    Summary:
    The value of the FileCheckNumericVariable class instance representing
    the @LINE numeric variable is set and cleared respectively before and
    after substitutions are made, if any. However, when a substitution
    fails, the value is not cleared. This causes the next substitution of
    @LINE later on to give the wrong value since setValue is a nop if the
    value is already set. This is what caused failures after commit r365249.

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

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

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64449 (detail)
    by thopre
  11. [NFC][ARM] Convert lambdas to static helpers

    Break up and convert some of the lambdas in ARMLowOverheadLoops into
    static functions. (detail)
    by sam_parker
  12. [X86] EltsFromConsecutiveLoads - LDBase is non-null. NFCI.

    Don't bother checking for LDBase != null - it should be (and we assert that it is). (detail)
    by rksimon
  13. [DAGCombine] visitINSERT_SUBVECTOR - use uint64_t subvector index. NFCI.

    Keep the uint64_t type from getZExtValue() to stop truncation/extension overflow warnings in MSVC in subvector index math. (detail)
    by rksimon
  14. [X86] EltsFromConsecutiveLoads - store Loads on a per-element basis. NFCI.

    Cache the LoadSDNode nodes so we can easily map to/from the element index instead of packing them together - this will be useful for future patches for PR16739 etc. (detail)
    by rksimon
  15. [ELF] Loose a condition for relocation with a symbol

    Deleted code was introduced as a work around for a bug in the gold linker
    (http://sourceware.org/PR16794). Test case that was given as a reason for
    this part of code, the one on previous link, now works for the gold.
    This condition is too strict and when a code is compiled with debug info
    it forces generation of numerous relocations with symbol for architectures
    that do not have relocation addend.

    Reviewers: arsenm, espindola

    Reviewed By: MaskRay

    Differential Revision: https://reviews.llvm.org/D64327 (detail)
    by nikolaprica
  16. [X86][SSE] EltsFromConsecutiveLoads - add basic dereferenceable support

    This patch checks to see if the vector element loads are based off a dereferenceable pointer that covers the entire vector width, in which case we don't need to have element loads at both extremes of the vector width - just the start (base pointer) of it.

    Another step towards partial vector loads......

    Differential Revision: https://reviews.llvm.org/D64205 (detail)
    by rksimon
  17. Fix const/non-const lambda return type warning. NFCI. (detail)
    by rksimon
  18. Fix "result of 32-bit shift implicitly converted to 64 bits" warning. NFCI. (detail)
    by rksimon
  19. [SimpleLoopUnswitch] Don't consider unswitching `switch` insructions with one unique successor

    Only instructions with two or more unique successors should be considered for unswitching.

    Patch Author: Daniil Suchkov.

    Reviewers: reames, asbirlea, skatkov
    Reviewed By: skatkov
    Subscribers: hiraditya, llvm-commits
    Differential Revision: https://reviews.llvm.org/D64404 (detail)
    by skatkov
  20. [AArch64] Fix vector vuqadd intrinsics operands

    Summary:
    Change the vuqadd vector instrinsics to have the second argument as unsigned values, not signed,
    accordingly to https://developer.arm.com/architectures/instruction-sets/simd-isas/neon/intrinsics

    Reviewers: LukeCheeseman, ostannard

    Reviewed By: ostannard

    Subscribers: javed.absar, kristof.beyls, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64211 (detail)
    by dnsampaio
  21. [NFC][AArch64] Fix vector vsqadd intrinsics operands

    Summary:
    Change the vsqadd vector instrinsics to have the second argument as signed values, not unsigned,
    accordingly to https://developer.arm.com/architectures/instruction-sets/simd-isas/neon/intrinsics

    Reviewers: LukeCheeseman, ostannard

    Reviewed By: ostannard

    Subscribers: javed.absar, kristof.beyls, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64210 (detail)
    by dnsampaio
  22. [clangd] Stop recording tokens before running clang-tidy

    modernize-trailing-return-type runs the preprocessor, breaking the token
    collection logic.

    This lead to a crash before, see the new test for a repro. (detail)
    by ibiryukov
  23. [clangd] Consume error to avoid assertion failures

    When we fail to calculate #include insertion for a completion item.
    Note that this change does not add a test, although that would be good. (detail)
    by ibiryukov
  24. [ARM] Enable VPUSH/VPOP aliases when either MVE or VFP is present

    Summary:
    Use the same predicates as VSTMDB/VLDMIA since VPUSH/VPOP alias to
    these.

    Patch by Momchil Velikov.

    Reviewers: ostannard, simon_tatham, SjoerdMeijer, samparker, t.p.northover, dmgreen

    Reviewed By: dmgreen

    Subscribers: javed.absar, kristof.beyls, hiraditya, dmgreen, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64413 (detail)
    by miyuki
  25. [docs][llvm-dwarfdump] Normalise some wording (detail)
    by jhenderson
  26. [clangd] Added highlighting for class and enum types.

    Summary: Added highlighting for non-builtin types using VisitTypeLoc. Ignoring namespace qualifiers as for now.

    Reviewers: hokein, sammccall, ilya-biryukov

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

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64257 (detail)
    by jvikstrom
  27. [test/Object/nm-trivial-object.test] - Remove 4 precompiled binaries.

    This converts 5 precompiled binaries to YAMLs,
    removes 4 from inputs and performs a cleanup.

    Differential revision: https://reviews.llvm.org/D64246 (detail)
    by grimar
  28. [SimpleLoopUnswitch] Add a test case exposing a bug

    This test exposes a bug in SimpleLoopUnswitch that leads to a crash on
    assert(SuccessorsCount > 1 && "Cannot unswitch a condition without multiple distinct successors!");
    when SimpleLoopUnswitch considers unswitching of a loop by a switch with one successor.

    Fix will be submitted soon.

    Patch Author: Daniil Suchkov.

    Reviewers: reames, asbirlea, skatkov
    Reviewed By: skatkov
    Subscribers: zzheng, llvm-commits
    Differential Revision: https://reviews.llvm.org/D64403 (detail)
    by skatkov
  29. [Syntax] Add assertion to catch invalid tokens early. NFC

    To help with identifiying root cause of a crash we are seeing. (detail)
    by ibiryukov
  30. [NFC][AArch64] Fix vector vqtb[lx][1-4]_s8 operand

    Summary:
    Change the vqtb[lx][1-4]_s8 instrinsics to have the last argument as vector of unsigned valuse, not
    signed, accordingly to https://developer.arm.com/architectures/instruction-sets/simd-isas/neon/intrinsics

    Reviewers: LukeCheeseman, DavidSpickett

    Reviewed By: DavidSpickett

    Subscribers: DavidSpickett, javed.absar, kristof.beyls, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64243 (detail)
    by dnsampaio
  31. Silence gcc warning by adding parentheses to condition [NFC]

    Without this gcc 7.4.0 complains with

      ../include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h:457:54: error: suggest parentheses around '&&' within '||' [-Werror=parentheses]
                        isArtifactCast(TmpDef->getOpcode()) &&
                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
                            "Expecting copy or artifact cast here");
                            ~ (detail)
    by uabelho

Started by upstream project clang-stage2-cmake-RgSan_relay build number 1175
originally caused by:

This run spent:

  • 2 ms waiting;
  • 10 hr build duration;
  • 10 hr total from scheduled to completion.
Test Result (no failures)

    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