Started 1 mo 17 days ago
Took 5 hr 40 min on green-dragon-02

Failed Build #14356 (Jul 8, 2019 3:02:29 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 365306
  • http://llvm.org/svn/llvm-project/cfe/trunk : 365305
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 365307
  • http://llvm.org/svn/llvm-project/zorg/trunk : 365252
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 365290
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 365289
Changes
  1. [ubsan][test] Fix cast-overflow.cpp and delete float-divide-by-zero test after D63793/rC365272 (detail)
    by maskray
  2. [ARM] Relax constraints on operands of VQxDMLxDH instructions

    Summary:
    According to a recently updated Armv8-M spec
    (https://static.docs.arm.com/ddi0553/bh/DDI0553B_h_armv8m_arm.pdf) the
    32-bit width versions of the following instructions:
    * VQDMLADH
    * VQDMLADHX
    * VQRDMLADH
    * VQRDMLADHX
    * VQDMLSDH
    * VQDMLSDHX
    * VQRDMLSDH
    * VQRDMLSDHX
    are no longer unpredictable when their output register is the same as
    one of the input registers.

    This patch updates the assembler parser and the corresponding tests
    and also removes @earlyclobber from the instruction constraints.

    Reviewers: simon_tatham, ostannard, dmgreen, SjoerdMeijer, samparker

    Reviewed By: simon_tatham

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

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64250 (detail)
    by miyuki
  3. [RISCV] Specify registers used for exception handling

    Implements the handling of __builtin_eh_return_regno().

    Differential Revision: https://reviews.llvm.org/D63417
    Patch by Edward Jones. (detail)
    by asb
  4. [ubsan][test] Fix several UBSan-* :: TestCases/ImplicitConversion tests on Solaris

    A couple of UBSan-*  :: TestCases/ImplicitConversion testcases FAIL on Solaris/x86
    (and Solaris/SPARC with https://reviews.llvm.org/D40900):

      FAIL: UBSan-AddressSanitizer-i386 :: TestCases/ImplicitConversion/signed-integer-truncation-or-sign-change-blacklist.c (49187 of 49849)
      ******************** TEST 'UBSan-AddressSanitizer-i386 :: TestCases/ImplicitConversion/signed-integer-truncation-or-sign-change-blacklist.c' FAILED ********************
      [...]
      Command Output (stderr):
      --
      /vol/llvm/src/compiler-rt/local/test/ubsan/TestCases/ImplicitConversion/signed-integer-truncation-or-sign-change-blacklist.c:53:11: error: CHECK: expected string not found in input
      // CHECK: {{.*}}signed-integer-truncation-or-sign-change-blacklist.c:[[@LINE-1]]:10: runtime error: implicit conversion from type '{{.*}}' (aka 'unsigned int') of value 4294967295 (32-bit, unsigned) to type '{{.*}}' (aka 'signed char') changed the value to -1 (8-bit, signed)
                ^
      <stdin>:1:1: note: scanning from here
      /vol/llvm/src/compiler-rt/local/test/ubsan/TestCases/ImplicitConversion/signed-integer-truncation-or-sign-change-blacklist.c:52:10: runtime error: implicit conversion from type 'uint32_t' (aka 'unsigned int') of value 4294967295 (32-bit, unsigned) to type 'int8_t' (aka 'char') changed the value to -1 (8-bit, signed)
      ^
      <stdin>:1:1: note: with "@LINE-1" equal to "52"
      /vol/llvm/src/compiler-rt/local/test/ubsan/TestCases/ImplicitConversion/signed-integer-truncation-or-sign-change-blacklist.c:52:10: runtime error: implicit conversion from type 'uint32_t' (aka 'unsigned int') of value 4294967295 (32-bit, unsigned) to type 'int8_t' (aka 'char') changed the value to -1 (8-bit, signed)
      ^
      <stdin>:1:69: note: possible intended match here
      /vol/llvm/src/compiler-rt/local/test/ubsan/TestCases/ImplicitConversion/signed-integer-truncation-or-sign-change-blacklist.c:52:10: runtime error: implicit conversion from type 'uint32_t' (aka 'unsigned int') of value 4294967295 (32-bit, unsigned) to type 'int8_t' (aka 'char') changed the value to -1 (8-bit, signed)
                                                                          ^

    This is always a difference for int8_t where signed char is expected, but only
    char seen.

    I could trace this to <sys/int_types.h> which has

      /*
       * Basic / Extended integer types
       *
       * The following defines the basic fixed-size integer types.
       *
       * Implementations are free to typedef them to Standard C integer types or
       * extensions that they support. If an implementation does not support one
       * of the particular integer data types below, then it should not define the
       * typedefs and macros corresponding to that data type.  Note that int8_t
       * is not defined in -Xs mode on ISAs for which the ABI specifies "char"
       * as an unsigned entity because there is no way to define an eight bit
       * signed integral.
       */
      #if defined(_CHAR_IS_SIGNED)
      typedef char int8_t;
      #else
      #if defined(__STDC__)
      typedef signed char int8_t;
      #endif
      #endif

    _CHAR_IS_SIGNED is always defined on both sparc and x86.  Since it seems ok
    to have either form, I've changed the affected tests to use
    '{{(signed )?}}char' instead of 'signed char'.

    Tested on x86_64-pc-solaris2.11, sparcv9-sun-solaris2.11, and x86_64-pc-linux-gnu.

    Differential Revision: https://reviews.llvm.org/D63984 (detail)
    by ro
  5. [ubsan][test] Don't disable ubsan testing on 64-bit Solaris/x86

    Unlike asan, which isn't supported yet on 64-bit Solaris/x86, there's no reason to disable
    ubsan.  This patch does that, but keeps the 64-bit ubsan-with-asan tests disabled.

    Tested on x86_64-pc-solaris2.11.

    Differential Revision: https://reviews.llvm.org/D63982 (detail)
    by ro
  6. [RISCV] Specify registers used in DWARF exception handling

    Defines RISCV registers for getExceptionPointerRegister() and
    getExceptionSelectorRegister().

    Differential Revision: https://reviews.llvm.org/D63411
    Patch by Edward Jones.
    Modified by Alex Bradbury to add CHECK lines to exception-pointer-register.ll. (detail)
    by asb
  7. [AArch64] Fix scalar vuqadd intrinsics operands

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

    So now the compiler correctly warns that a undefined negative float conversion is being done.

    Reviewers: LukeCheeseman, john.brawn

    Reviewed By: john.brawn

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

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64242 (detail)
    by dnsampaio
  8. [ARM] Fix null pointer dereference in CodeGen/ARM/Windows/stack-protector-msvc.ll.test after D64292/r365283

    CLI.CS may not be set. (detail)
    by maskray
  9. [AArch64] Fix vsqadd scalar intrinsics operands

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

    The existing unsigned argument can cause faulty code as negative float to unsigned conversion is
    undefined, which llvm/clang optimizes away.

    Reviewers: LukeCheeseman, john.brawn

    Reviewed By: john.brawn

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

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64239 (detail)
    by dnsampaio
  10. [UpdateTestChecks] Skip over .Lfunc_begin for RISC-V

    This mirrors the change made for X86 in rL336987. Without this patch,
    update_llc_test_checks will completely skip functions with personality
    functions. (detail)
    by asb
  11. [AMDGPU] Use a named predicate instead of a magic number.

    Reviewers: arsenm

    Reviewed By: arsenm

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

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64201 (detail)
    by foad
  12. [X86] Allow execution domain fixing to turn SHUFPD into SHUFPS.

    This can help with code size on SSE targets where SHUFPD requires
    a 0x66 prefix and SHUFPS doesn't. (detail)
    by ctopper
  13. [X86] Make movsd commutable to shufpd with a 0x02 immediate on pre-SSE4.1 targets.

    This can help avoid a copy or enable load folding.

    On SSE4.1 targets we can commute it to blendi instead.

    I had to make shufpd with a 0x02 immediate commutable as well
    since we expect commuting to be reversible. (detail)
    by ctopper

Started by timer (4 times)

This run spent:

  • 3 hr 51 min waiting;
  • 5 hr 40 min build duration;
  • 9 hr 31 min total from scheduled to completion.
Test Result (5 failures / +3)

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