Started 4 mo 1 day ago
Took 55 min

Success Build clang-r361516-t56783-b56783.tar.gz (May 23, 2019 11:25:11 AM)

Issues

No known issues detected

Build Log

Revision: 358206
Changes
  1. [WebAssembly] Add multivalue and tail-call target features

    Summary:
    These features will both be implemented soon, so I thought I would
    save time by adding the boilerplate for both of them at the same time.

    Reviewers: aheejin

    Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, cfe-commits, llvm-commits

    Tags: #clang, #llvm

    Differential Revision: https://reviews.llvm.org/D62047 (detail)
    by tlively
  2. [FileCheck] Remove llvm:: prefix

    Summary:
    Remove all llvm:: prefixes in FileCheck library header and
    implementation except for calls to make_unique and make_shared since
    both files already use the llvm namespace.

    Reviewers: jhenderson, jdenny, probinson, arichardson

    Subscribers: hiraditya, arichardson, probinson, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D62323 (detail)
    by thopre
  3. [CMake] Copy C++ headers before configuring runtimes build

    Summary: On some platforms C++ headers are packaged with the compiler not the sysroot. If you don't copy C++ headers into the build include directory during configuraiton of the outer build the C++ check during the runtime configuration may get inaccurate results.

    Reviewers: phosek, compnerd, smeenai, EricWF

    Reviewed By: compnerd

    Subscribers: EricWF, christof, libcxx-commits, mgorny, llvm-commits

    Tags: #llvm, #libc

    Differential Revision: https://reviews.llvm.org/D62155 (detail)
    by cbieneman
  4. Transforms: lower fadd and fsub atomicrmw instructions

    `fadd` and `fsub` have recently (r351850) been added as `atomicrmw`
    operations. This diff adds lowering cases for them to the LowerAtomic
    transform.

    Patch by Josh Berdine! (detail)
    by Saleem Abdulrasool
  5. [MCA] Add the ability to compute critical register dependency of an instruction.

    This patch adds the methods `getCriticalRegDep()` and `computeCriticalRegDep()` to
    class InstructionBase.
    The goal is to allow users to obtain information about the critical register
    dependency that most affects the latency of an instruction.

    These methods are currently unused. However, the long term plan is to use them
    in order to allow the computation of a critical-path as part of the bottleneck
    analysis. So, this is yet another step towards fixing PR37494. (detail)
    by adibiagio
  6. [AsmPrinter] Treat a narrowing PtrToInt like Trunc

    When printing assembly for PtrToInt, AsmPrinter::lowerConstant
    incorrectly assumed that if PtrToInt was not converting to an
    int with exactly the same number of bits, it must be widening
    to a larger int. But this isn't necessarily true; PtrToInt can
    also shrink the size, which is useful when you want to produce
    a known 32-bit pointer on a 64-bit platform (on x86_64 ELF
    this yields a R_X86_64_32 relocation).

    The old behavior of falling through to the widening case for a
    narrowing PtrToInt yields bogus assembly code like this, which
    fails to assemble because the no-op bit and it accidentally
    creates is not a valid relocation:

    ```
            .long   a&-1
    ```

    The fix is to treat a narrowing PtrToInt exactly the same as
    it already treats Trunc: just emit the expression and let
    the assembler deal with truncating it in the appropriate way.

    Patch by Mat Hostetter <mjh@fb.com>.

    Differential Revision: https://reviews.llvm.org/D61325 (detail)
    by smeenai
Revision: 358206
Changes
  1. [WebAssembly] Add multivalue and tail-call target features

    Summary:
    These features will both be implemented soon, so I thought I would
    save time by adding the boilerplate for both of them at the same time.

    Reviewers: aheejin

    Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, cfe-commits, llvm-commits

    Tags: #clang, #llvm

    Differential Revision: https://reviews.llvm.org/D62047 (detail)
    by tlively
  2. [LibTooling] Fix dangling references in RangeSelector.

    Summary:
    RangeSelector had a number of cases of capturing a StringRef in a lambda, which
    lead to dangling references. This change converts all uses in the API of
    `StringRef` to `std::string` to avoid this problem. `std::string` in the API is
    a reasonable choice, because the combinators are always storing the string
    beyond the life of the combinator construction.

    Reviewers: ilya-biryukov, gribozavr

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D62328 (detail)
    by ymandel
  3. [Index] Fix reported references in presence of template type aliases

    Summary: See the added test for an example.

    Reviewers: kadircet

    Reviewed By: kadircet

    Subscribers: jkorous, arphaman, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D62303 (detail)
    by ibiryukov
  4. [CodeComplete] Only show lambda completions if patterns are requested

    This is a trivial follow-up to r361461, so sending without review. (detail)
    by ibiryukov
  5. Ensure builtins use the target default Calling Convention

    r355317 changed builtins/allocation functions to use the default calling
    convention in order to support platforms that use non-cdecl calling
    conventions by default.

    However the default calling convention is overridable on Windows 32 bit
    implementations with some of the /G options. The intent is to permit the
    user to set the calling convention of normal functions, however it
    should NOT apply to builtins and C++ allocation functions.

    This patch ensures that the builtin/allocation functions always use the
    Target specific Calling Convention, ignoring the user overridden version
    of said default. (detail)
    by erichkeane
Revision: 358206
Changes
  1. [Index] Fix reported references in presence of template type aliases

    Summary: See the added test for an example.

    Reviewers: kadircet

    Reviewed By: kadircet

    Subscribers: jkorous, arphaman, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D62303 (detail)
    by ibiryukov
Revision: 358206
Changes
  1. [CMake] Copy C++ headers before configuring runtimes build

    Summary: On some platforms C++ headers are packaged with the compiler not the sysroot. If you don't copy C++ headers into the build include directory during configuraiton of the outer build the C++ check during the runtime configuration may get inaccurate results.

    Reviewers: phosek, compnerd, smeenai, EricWF

    Reviewed By: compnerd

    Subscribers: EricWF, christof, libcxx-commits, mgorny, llvm-commits

    Tags: #llvm, #libc

    Differential Revision: https://reviews.llvm.org/D62155 (detail)
    by cbieneman

Started by upstream project relay-test-suite-verify-machineinstrs build number 5206
originally caused by:

This run spent:

  • 43 min waiting;
  • 55 min build duration;
  • 56 min total from scheduled to completion.