Started 5 mo 24 days ago
Took 28 min

Success Build clang-r361484-t56770-b56770.tar.gz (May 23, 2019 5:04:56 AM)

Issues

No known issues detected

Build Log

Revision: 358206
Changes
  1. [lldb] NFC modernize codebase with modernize-use-nullptr

    Summary:
    NFC = [[ https://llvm.org/docs/Lexicon.html#nfc | Non functional change ]]

    This commit is the result of modernizing the LLDB codebase by using
    `nullptr` instread of `0` or `NULL`. See
    https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-nullptr.html
    for more information.

    This is the command I ran and I to fix and format the code base:

    ```
    run-clang-tidy.py \
    -header-filter='.*' \
    -checks='-*,modernize-use-nullptr' \
    -fix ~/dev/llvm-project/lldb/.* \
    -format \
    -style LLVM \
    -p ~/llvm-builds/debug-ninja-gcc
    ```

    NOTE: There were also changes to `llvm/utils/unittest` but I did not
    include them because I felt that maybe this library shall be updated in
    isolation somehow.

    NOTE: I know this is a rather large commit but it is a nobrainer in most
    parts.

    Reviewers: martong, espindola, shafik, #lldb, JDevlieghere

    Reviewed By: JDevlieghere

    Subscribers: arsenm, jvesely, nhaehnle, hiraditya, JDevlieghere, teemperor, rnkovacs, emaste, kubamracek, nemanjai, ki.stfu, javed.absar, arichardson, kbarton, jrtc27, MaskRay, atanasyan, dexonsmith, arphaman, jfb, jsji, jdoerfert, lldb-commits, llvm-commits

    Tags: #lldb, #llvm

    Differential Revision: https://reviews.llvm.org/D61847 (detail)
    by kwk
  2. [NFC][X86] Fix check prefixes and autogenerate fold-pcmpeqd-2.ll test

    Being affected by (sub %x, c) -> (add %x, (sub 0, c))
    patch in an uncertain way. (detail)
    by lebedevri
  3. [MCA] Make the bool conversion operator in class InstRef explicit. NFCI

    This patch makes the bool conversion operator in InstRef explicit.
    It also adds a operator< to hel comparing InstRef objects in sets. (detail)
    by adibiagio
  4. [DwarfExpression] Refactor dwarf expression (NFC)

    Refactor location description kind in order to be easier for extensions
    (needed for D60866).
    In addition, cut off some bits from the other class fields.

    Patch by Djordje Todorovic.

    Differential Revision: https://reviews.llvm.org/D62002 (detail)
    by petarj
  5. [llvm-objdump][test] Improve testing of some switches #2

    This patch focuses on adding additional testing for the --source switch.
    For reference, the source-interleave-x86_64.ll test file has been split
    into two parts - the input (shared with the other tests) and the test
    itself.

    Reviewed by: MaskRay, rupprecht, grimar

    Differential Revision: https://reviews.llvm.org/D61996 (detail)
    by jhenderson
Revision: 358206
Changes
  1. Delete default constructors, copy constructors, move constructors, copy assignment, move assignment operators on Expr, Stmt and Decl

    Reviewers: ilya-biryukov, rsmith

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D62187 (detail)
    by gribozavr
  2. Enable queue_t and clk_event_t comparisons in C++ mode

    Support queue_t and clk_event_t comparisons in C++ for OpenCL mode, to
    preserve backwards compatibility with OpenCL C.

    Differential Revision: https://reviews.llvm.org/D62208 (detail)
    by svenvh
Revision: 358206
Changes
  1. [clangd-vscode] Do not customize uri converters in vscode

    Summary:
    Clangd is already resolving symlinks on the server side, therefore
    there is no more need to handle it in client side. This was also resulting in
    breakages whenever index contained a symbol coming from a non-existent file(like
    a generated file), e.g. during workspace symbols whole response was dropped
    since stat had failed.

    Reviewers: ilya-biryukov

    Subscribers: MaskRay, jkorous, arphaman, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D62288 (detail)
    by kadircet

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

This run spent:

  • 8 min 6 sec waiting;
  • 28 min build duration;
  • 28 min total from scheduled to completion.