Started 9 days 13 hr ago
Took 4 hr 48 min on green-dragon-20

Failed Build #18431 (Jul 11, 2019 8:12:53 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 365779
  • http://llvm.org/svn/llvm-project/cfe/trunk : 365777
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 365739
  • 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 : 365778
Changes
  1. Revert r365775 - "[Object/ELF] - Improve error reporting for notes."

    It broke BB: http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/15419 (detail/ViewSVN)
    by grimar
  2. [clangd] Remove an extra ";", NFC (detail/ViewSVN)
    by hokein
  3. Added mapping for pointers captured in Lambdas in OpenMP target regions,
    by David Truby.

    Summary:
    This adds a zero length array section mapping for each pointer captured by a lambda that is used in a target region, as per section 2.19.7.1 of the OpenMP 5 specification.

    Reviewers: ABataev

    Reviewed By: ABataev

    Subscribers: guansong, jdoerfert, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64558 (detail/ViewSVN)
    by abataev
  4. [clangd][QueryDriver] Use language from underlying database if possible

    Reviewers: sammccall

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

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64489 (detail/ViewSVN)
    by kadircet
  5. [Object/ELF] - Improve error reporting for notes.

    This patch improves the error messages reported for
    note sections and phdrs and also makes a cleanup for
    existent test case.

    Differential revision: https://reviews.llvm.org/D64470 (detail/ViewSVN)
    by grimar
  6. [InstCombine] don't move FP negation out of a constant expression

    -(X * ConstExpr) becomes X * (-ConstExpr), so don't reverse that
    and infinite loop. (detail/ViewSVN)
    by spatel
  7. [clangd] Add priorities to background index queue, extract to separate class

    Reviewers: kadircet

    Subscribers: mgorny, ilya-biryukov, MaskRay, jkorous, arphaman, jfb, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64560 (detail/ViewSVN)
    by sammccall
  8. [yaml2obj/elf-override-shsize.yaml] - An attemp to fix ppc64 bot.

    Failture:
    http://lab.llvm.org:8011/builders/clang-ppc64be-linux/builds/35670/steps/ninja%20check%201/logs/FAIL%3A%20LLVM%3A%3Aelf-override-shsize.yaml

    Solution:
    Change `od` tool invocation to print single bytes. (detail/ViewSVN)
    by grimar
  9. OpaquePtr: switch to GlobalValue::getValueType in a few places. NFC. (detail/ViewSVN)
    by Tim Northover
  10. OpaquePtr: use byval accessor instead of inspecting pointer type. NFC.

    The accessor can deal with both "byval(ty)" and "ty* byval" forms
    seamlessly. (detail/ViewSVN)
    by Tim Northover
  11. OpaquePtr: use load instruction directly for type. NFC. (detail/ViewSVN)
    by Tim Northover
  12. InstructionSimplify: Simplify InstructionSimplify. NFC.

    The interface predates CallBase, so both it and implementation were
    significantly more complicated than they needed to be. There was even
    some redundancy that could be eliminated.

    Should also help with OpaquePointers by not trying to derive a
    function's type from it's PointerType. (detail/ViewSVN)
    by Tim Northover
  13. [yaml2obj] - Allow overriding the sh_size field.

    There is no way to set broken sh_size field currently
    for sections. It can be usefull for writing the
    test cases.

    Differential revision: https://reviews.llvm.org/D64401 (detail/ViewSVN)
    by grimar
  14. [NFC] Updated tests for D64285 (detail/ViewSVN)
    by xbolva00
  15. [clangd] Remove the extra ";", NFC (detail/ViewSVN)
    by hokein
  16. [clangd] Fix an assertion crash in "ExtractVariable" tweak

    Summary:
    GetTypePtr requires that the type should not be null, otherwise we hit
    an assertion, we should use getTypePtrOrNull instead.

    Reviewers: sammccall, SureYeaah

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

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64556 (detail/ViewSVN)
    by hokein
  17. [llvm-readobj/llvm-readelf] - Report a warning instead of a error when dumping a broken dynamic section.

    It does not make sence to stop dumping the object if the broken
    dynamic section was found. In this patch I changed the behavior from
    "report an error" to "report a warning". This matches GNU.

    Differential revision: https://reviews.llvm.org/D64472 (detail/ViewSVN)
    by grimar
  18. [InstCombine] Reorder recently added/improved pow transformations

    Changed cases are now faster with exp2. (detail/ViewSVN)
    by xbolva00
  19. Revert [BitcodeReader] Validate OpNum, before accessing Record array.

    This reverts r365750 (git commit 8b222ecf2769ee133691f208f6166ce118c4a164)

    llvm-dis runs out of memory while opening invalid-fcmp-opnum.bc on
    llvm-hexagon-elf, probably because the bitcode file contains other
    suspicious values.

    http://lab.llvm.org:8011/builders/llvm-hexagon-elf/builds/21949 (detail/ViewSVN)
    by fhahn
  20. [clangd] Fix windows buildbots (detail/ViewSVN)
    by kadircet
  21. [X86] Regenerate intrinsics tests. NFCI. (detail/ViewSVN)
    by rksimon
  22. [AMDGPU] Regenerate idot tests. NFCI.

    Reduces diff in D63281. (detail/ViewSVN)
    by rksimon
  23. [llvm-objcopy] Don't change permissions of non-regular output files

    There is currently an EPERM error when a regular user executes `llvm-objcopy a.o /dev/null`.
    Worse, root can even change the mode bits of /dev/null.

    Fix it by checking if the output file is special.

    A new overload of llvm::sys::fs::setPermissions with FD as the parameter
    is added. Users should provide `perm & ~umask` as the parameter if they
    intend to respect umask.

    The existing overload of llvm::sys::fs::setPermissions may be deleted if
    we can find an implementation of fchmod() on Windows. fchmod() is
    usually better than chmod() because it saves syscalls and can avoid race
    condition.

    Reviewed By: jakehehrlich, jhenderson

    Differential Revision: https://reviews.llvm.org/D64236 (detail/ViewSVN)
    by maskray
  24. [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/ViewSVN)
    by maskray
  25. [clangd] Add a missing "return nullptr;" to the SelectionTree::commonAncestor.

    This would fix some mysterious crashes on codeAction in clangd. (detail/ViewSVN)
    by hokein
  26. [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/ViewSVN)
    by fhahn
  27. [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/ViewSVN)
    by sam_parker
  28. [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/ViewSVN)
    by kadircet
  29. [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/ViewSVN)
    by statham
  30. [NFC][PowerPC] Added test to track current behaviour of TailDup (detail/ViewSVN)
    by lkail
  31. [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/ViewSVN)
    by jvikstrom

Started by upstream project clang-stage2-Rthinlto_relay build number 1840
originally caused by:

This run spent:

  • 1 ms waiting;
  • 4 hr 48 min build duration;
  • 4 hr 48 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