Started 1 mo 9 days ago
Took 5 hr 12 min on green-dragon-02

Failed Build #14367 (Jul 10, 2019 9:55:49 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 365663
  • http://llvm.org/svn/llvm-project/cfe/trunk : 365643
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 365591
  • http://llvm.org/svn/llvm-project/zorg/trunk : 365596
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 365562
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 365634
Changes
  1. [NFC][InstCombine] Fixup some tests in just-added "omit mask before left-shift" tests (detail)
    by lebedevri
  2. MC: AArch64: Add support for pg_hi21_nc relocation specifier.

    Differential Revision: https://reviews.llvm.org/D64455 (detail)
    by pcc
  3. [CodeExtractor] Fix sinking of allocas with multiple bitcast uses (PR42451)

    An alloca which can be sunk into the extraction region may have more
    than one bitcast use. Move these uses along with the alloca to prevent
    use-before-def.

    Testing: check-llvm, stage2 build of clang

    Fixes llvm.org/PR42451.

    Differential Revision: https://reviews.llvm.org/D64463 (detail)
    by Vedant Kumar
  4. [CodeExtractor] Simplify findAllocas, NFC

    Split getLifetimeMarkers out into its own method and have it return a
    struct.

    Differential Revision: https://reviews.llvm.org/D64467 (detail)
    by Vedant Kumar
  5. GlobalISel: Legalization for G_FMINNUM/G_FMAXNUM (detail)
    by arsenm
  6. GlobalISel: Define the full family of FP min/max instructions (detail)
    by arsenm
  7. [X86] EltsFromConsecutiveLoads - remove duplicate check for element size. NFCI.

    We've already checked that each element is the correct contributory size for VT when we inspect the elements for Undef/Zero/Load. (detail)
    by rksimon
  8. [X86] EltsFromConsecutiveLoads - ensure element reg/store sizes are the same size. NFCI.

    This renames the type so it doesn't sound like its based off the load size - as we're moving towards supporting combining loads of different sizes. (detail)
    by rksimon
  9. AMDGPU: Serialize mode from MachineFunctionInfo (detail)
    by arsenm
  10. [PatternMatch] Generalize m_SpecificInt_ULT() to take ICmpInst::Predicate

    As discussed in the original review, this may be useful,
    so let's just do it. (detail)
    by lebedevri
  11. [Remarks] Add cl::Hidden to -remarks-yaml-string-table

    It was showing up in a lot of unrelated tools. (detail)
    by thegameg
  12. docs/GithubMove.rst: Remove obsolete information

    Summary:
    Remove references to the multirepo and update the document to
    reflect the current state of the github repository.

    Reviewers: mehdi_amini, jyknight

    Subscribers: jdoerfert, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D58420 (detail)
    by tstellar
  13. [FileCheck] Use bool operator to test Expected

    Use bool() consistently to get boolean value of Error, Optional and
    Expected types in EXPECT calls. While static_cast is used in all cases
    but one, bool provides more clarity and makes more sense as a new
    default. (detail)
    by thopre
  14. [AMDGPU] Increased the number of implicit argument bytes for both OpenCL and HIP (CLANG).

    To enable a new implicit kernel argument,
    increased the number of argument bytes from 48 to 56.

    Reviewed By: yaxunl

    Differential Revision: https://reviews.llvm.org/D63756 (detail)
    by cdevadas
  15. [NFC][InstCombine] Redundant masking before left-shift (PR42563)

    alive proofs:
    a,b:     https://rise4fun.com/Alive/4zsf
    c,d,e,f: https://rise4fun.com/Alive/RC49

    Indeed, not all of these patterns are canonical.
    But since this fold will only produce a single instruction
    i'm really interested in handling even uncanonical patterns.

    Other than these 6 patterns, i can't think of any other
    reasonable variants right now, although i'm sure they exist.

    For now let's start with patterns where both shift amounts are variable,
    with trivial constant "offset" between them, since i believe this is
    both simplest to handle and i think this is most common.
    But again, there are likely other variants where we could use
    ValueTracking/ConstantRange to handle more cases.

    https://bugs.llvm.org/show_bug.cgi?id=42563 (detail)
    by lebedevri
  16. [AMDGPU] Allow abs/neg source modifiers on v_cndmask_b32

    Summary:
    D59191 added support for these modifiers in the assembler and
    disassembler. This patch just teaches instruction selection that it can
    use them.

    Reviewers: arsenm, tstellar

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

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64497 (detail)
    by foad
  17. [analyzer]Add user docs rst

    Summary:
    Add user documentation page. This is an empty page atm, later patches will add
    the specific user documentatoins.

    Reviewers: dkrupp

    Subscribers: whisperity, xazax.hun, baloghadamsoftware, szepet, rnkovacs, a.sidorin, mikhail.ramalho, Szelethus, donat.nagy, gamesh411, Charusso, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64494 (detail)
    by martong
  18. Remove two unused member variables.

    They were added over 10 years ago in r66575 and have never been used as
    far as I can tell.

    (r67087 added similar fields to Compilation, and those are used.) (detail)
    by nico
  19. [InstCombine] pow(C,x) -> exp2(log2(C)*x)

    Summary:
    Transform
    pow(C,x)

    To
    exp2(log2(C)*x)

    if C > 0, C != inf, C != NaN (and C is not power of 2, since we have some fold for such case already).

    log(C) is folded by the compiler and exp2 is much faster to compute than pow.

    Reviewers: spatel, efriedma, evandro

    Reviewed By: evandro

    Subscribers: lebedev.ri, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64099 (detail)
    by xbolva00
  20. [InferFunctionAttrs] add/adjust tests for dereferenceable; NFC

    Based on review comments for D64258. (detail)
    by spatel
  21. [clangd] Filter out non-governed files from broadcast

    Summary:
    This also turns off implicit discovery of additional compilation
    databases.

    Reviewers: sammccall

    Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64247 (detail)
    by kadircet
  22. gn build: Merge r365585 (detail)
    by nico
  23. [clangd] Trim spaces around parsed include in include extractor (detail)
    by kadircet
  24. [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
  25. [docs][llvm-symbolizer] Fix grammar (detail)
    by jhenderson
  26. 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
  27. [X86] EltsFromConsecutiveLoads - cleanup Zero/Undef/Load element collection. NFCI. (detail)
    by rksimon
  28. [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
  29. [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
  30. [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
  31. [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
  32. [NFC][ARM] Convert lambdas to static helpers

    Break up and convert some of the lambdas in ARMLowOverheadLoops into
    static functions. (detail)
    by sam_parker
  33. [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
  34. [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
  35. [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
  36. [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
  37. [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
  38. Fix const/non-const lambda return type warning. NFCI. (detail)
    by rksimon
  39. Fix "result of 32-bit shift implicitly converted to 64 bits" warning. NFCI. (detail)
    by rksimon

Started by timer (6 times)

This run spent:

  • 5 hr 44 min waiting;
  • 5 hr 12 min build duration;
  • 10 hr total from scheduled to completion.
Test Result (5 failures / ±0)Show all failed tests >>>

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

Unexpected pass

This buildbot failed because a test marked as XFAIL unexpectedly passes. This could mean that the cause for the XFAIL is fixed, but it warrants investigation in any case.
Indication 2