Started 8 days 19 hr ago
Took 3 hr 54 min on green-dragon-02

Failed Build #14371 (Jul 11, 2019 3:10:50 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 365752
  • http://llvm.org/svn/llvm-project/cfe/trunk : 365742
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 365739
  • 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 : 365751
Changes
  1. [X86] -fno-plt: use GOT __tls_get_addr only if GOTPCRELX is enabled

    Summary:
    As of binutils 2.32, ld has a bogus TLS relaxation error when the GD/LD
    code sequence using R_X86_64_GOTPCREL (instead of R_X86_64_GOTPCRELX) is
    attempted to be relaxed to IE/LE (binutils PR24784). gold and lld are good.

    In gcc/config/i386/i386.md, there is a configure-time check of as/ld
    support and the GOT relaxation will not be used if as/ld doesn't support
    it:

        if (flag_plt || !HAVE_AS_IX86_TLS_GET_ADDR_GOT)
          return "call\t%P2";
        return "call\t{*%p2@GOT(%1)|[DWORD PTR %p2@GOT[%1]]}";

    In clang, -DENABLE_X86_RELAX_RELOCATIONS=OFF is the default. The ld.bfd
    bogus error can be reproduced with:

        thread_local int a;
        int main() { return a; }

    clang -fno-plt -fpic a.cc -fuse-ld=bfd

    GOTPCRELX gained relative good support in 2016, which is considered
    relatively new.  It is even difficult to conditionally default to
    -DENABLE_X86_RELAX_RELOCATIONS=ON due to cross compilation reasons. So
    work around the ld.bfd bug by only using GOT when GOTPCRELX is enabled.

    Reviewers: dalias, hjl.tools, nikic, rnk

    Reviewed By: nikic

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64304 (detail)
    by maskray
  2. [clangd] Add a missing "return nullptr;" to the SelectionTree::commonAncestor.

    This would fix some mysterious crashes on codeAction in clangd. (detail)
    by hokein
  3. [BitcodeReader] Validate OpNum, before accessing Record array.

    Currently invalid bitcode files can cause a crash, when OpNum exceeds
    the number of elements in Record, like in the attached bitcode file.

    The test case was generated by clusterfuzz: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=15698

    Reviewers: t.p.northover, thegameg, jfb

    Reviewed By: jfb

    Differential Revision: https://reviews.llvm.org/D64507 (detail)
    by fhahn
  4. [ARM][LowOverheadLoops] Correct offset checking
       
    This patch addresses a couple of problems:
    1) The maximum supported offset of LE is -4094.
    2) The offset of WLS also needs to be checked, this uses a
       maximum positive offset of 4094.
       
    The use of BasicBlockUtils has been changed because the block offsets
    weren't being initialised, but the isBBInRange checks both positive
    and negative offsets.
       
    ARMISelLowering has been tweaked because the test case presented
    another pattern that we weren't supporting. (detail)
    by sam_parker
  5. [clangd] Reland rL365634

    This was reverted in rL365678, the failure was due to YAML parsing of
    compile_commands.json.

    Converting backslashes to forward slashes to fix the issue in unittest. (detail)
    by kadircet
  6. [ARM] Remove nonexistent unsigned forms of MVE VQDMLAH.

    The VQDMLAH.U8, VQDMLAH.U16 and VQDMLAH.U32 instructions don't
    actually exist: the Armv8.1-M architecture spec only lists signed
    forms of that instruction. The unsigned ones were added in error: they
    existed in an early draft of the spec, but they were removed before
    the public version, and we missed that particular spec change.

    Also affects the variant forms VQDMLASH, VQRDMLAH and VQRDMLASH.

    Reviewers: miyuki

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

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64502 (detail)
    by statham
  7. [NFC][PowerPC] Added test to track current behaviour of TailDup (detail)
    by lkail
  8. [clangd] Added highlightings for namespace specifiers.

    Summary: Added highlightings for namespace specifiers.

    Reviewers: hokein, sammccall, ilya-biryukov

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

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64492 (detail)
    by jvikstrom
  9. [MIPS GlobalISel] Skip copies in addUseDef and addDefUses

    Skip copies between virtual registers during search for UseDefs
    and DefUses.
    Since each operand has one def search for UseDefs is straightforward.
    But since operand can have many uses, we have to check all uses of
    each copy we traverse during search for DefUses.

    Differential Revision: https://reviews.llvm.org/D64486 (detail)
    by petar.avramovic
  10. [MIPS GlobalISel] RegBankSelect for chains of ambiguous instructions

    When one of the uses/defs of ambiguous instruction is also ambiguous
    visit it recursively and search its uses/defs for instruction with
    only one mapping available.
    When all instruction in a chain are ambiguous arbitrary mapping can
    be selected. For s64 operands in ambiguous chain fprb is selected since
    it results in less instructions then having to narrow scalar s64 to s32.
    For s32 both gprb and fprb result in same number of instructions and
    gprb is selected like a general purpose option.

    At the moment we always avoid cross register bank copies.
    TODO: Implement a model for costs calculations of different mappings
    on same instruction and cross bank copies. Allow cross bank copies
    when appropriate according to cost model.

    Differential Revision: https://reviews.llvm.org/D64485 (detail)
    by petar.avramovic
  11. Revert Recommit "[CommandLine] Remove OptionCategory and SubCommand caches from the Option class."

    This reverts r365675 (git commit 43d75f977853c3ec891a440c362b2df183a211b5)

    The patch causes a crash in SupportTests (CommandLineTest.AliasesWithArguments). (detail)
    by hokein
  12. Remove some redundant code from r290372 and improve a comment. (detail)
    by foad
  13. [ARM][ParallelDSP] Change the search for smlads
       
    Two functional changes have been made here:
    - Now search up from any add instruction to find the chains of
      operations that we may turn into a smlad. This allows the
      generation of a smlad which doesn't accumulate into a phi.
    - The search function has been corrected to stop it falsely searching
      up through an invalid path.
       
    The bulk of the changes have been making the Reduction struct a class
    and making it more C++y with getters and setters.

    Differential Revision: https://reviews.llvm.org/D61780 (detail)
    by sam_parker
  14. [ASan] Use __sanitizer_fill_shadow for FastPoisonShadow on Fuchsia

    This is the optimized implementation for Fuchsia provided by the libc.

    Differential Revision: https://reviews.llvm.org/D64166 (detail)
    by phosek
  15. [test] Silence gcc 7.4 warning [NFC]

    Without this gcc 7.4.0 complains with
    ../unittests/Analysis/ValueTrackingTest.cpp:937:66: error: ISO C++11 requires at least one argument for the "..." in a variadic macro [-Werror]
                              ::testing::ValuesIn(IsBytewiseValueTests));
                                                                       ^ (detail)
    by uabelho
  16. [clang-doc] Silence compiler warning with gcc 7.4 [NFC]

    Without the fix gcc 7.4.0 complains with

    /data/repo/master/clang-tools-extra/clang-doc/HTMLGenerator.cpp: In member function 'llvm::SmallString<16> clang::doc::{anonymous}::HTMLTag::ToString() const':
    /data/repo/master/clang-tools-extra/clang-doc/HTMLGenerator.cpp:165:1: error: control reaches end of non-void function [-Werror=return-type]
    }
    ^
    /data/repo/master/clang-tools-extra/clang-doc/HTMLGenerator.cpp: In member function 'bool clang::doc::{anonymous}::HTMLTag::HasInlineChildren() const':
    /data/repo/master/clang-tools-extra/clang-doc/HTMLGenerator.cpp:142:1: error: control reaches end of non-void function [-Werror=return-type]
    }
    ^
    /data/repo/master/clang-tools-extra/clang-doc/HTMLGenerator.cpp: In member function 'bool clang::doc::{anonymous}::HTMLTag::IsSelfClosing() const':
    /data/repo/master/clang-tools-extra/clang-doc/HTMLGenerator.cpp:126:1: error: control reaches end of non-void function [-Werror=return-type]
    }
    ^
    cc1plus: all warnings being treated as errors (detail)
    by uabelho
  17. Add NetBSD LSan support

    Summary:
    Combine few relatively small changes into one:

    - implement internal_ptrace() and internal_clone() for NetBSD
    - add support for stoptheworld based on the ptrace(2) API
    - define COMPILER_RT_HAS_LSAN for NetBSD
    - enable tests for NetBSD/amd64

    Inspired by the original implementation by Christos Zoulas in netbsd/src for GCC.

    The implementation is in theory CPU independent through well defined macros
    across all NetBSD ports, however only the x86_64 version was tested.

    Reviewers: mgorny, dvyukov, vitalybuka, joerg, jfb

    Reviewed By: vitalybuka

    Subscribers: dexonsmith, jfb, srhines, kubamracek, llvm-commits, christos

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64057 (detail)
    by kamil

Started by timer (4 times)

This run spent:

  • 3 hr 0 min waiting;
  • 3 hr 54 min build duration;
  • 6 hr 55 min total from scheduled to completion.

Identified problems

Ninja target failed

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

Missing test results

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