Started 8 mo 10 days ago
Took 24 min on green-dragon-13

Failed Build #279 (Feb 8, 2019 10:30:05 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 353539
  • http://llvm.org/svn/llvm-project/cfe/trunk : 353542
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 353528
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 346271
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 353450
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 353535
Changes
  1. [OPENMP]Delay emission of the error messages for the exceptions.

    Fixed diagnostic emission for the exceptions support in case of the
    compilation of OpenMP code for the devices. From now on, it uses delayed
    diagnostics mechanism, previously used for CUDA only. It allow to
    diagnose not allowed used of exceptions only in functions that are going
    to be codegen'ed. (detail/ViewSVN)
    by abataev
  2. Revert "[OPENMP]Initial support for the delayed diagnostics."

    This reverts commit r353540. Erroneously committed, need to fix the
    message and description. (detail/ViewSVN)
    by abataev
  3. [OPENMP]Initial support for the delayed diagnostics.

    It is important to delay the emission of the diagnostic messages for the
    functions unless it is proved that the function is going to be used on
    the device side. It is required to support compilation with some of the
    target-specific system headers. (detail/ViewSVN)
    by abataev
  4. [TargetLowering] Add SimplifyDemandedBits funnel shift support (detail/ViewSVN)
    by rksimon
  5. ArgumentPromotion should copy all metadata to new Function

    Summary:
    ArgumentPromotion had code to specifically move the dbg metadata over to
    the new function, but other metadata such as the function_entry_count
    !prof metadata was not. Replace code that moved dbg metadata with a call
    to copyMetadata. The old metadata is automatically removed when the old
    Function is removed.

    Reviewers: davidxl

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D57846 (detail/ViewSVN)
    by tejohnson
  6. [X86] Remove isReMaterializable from X87 floating point constant loads and constant pool loads.

    Summary: These instructions update FPSW so they aren't generically safe to rematerialize into any location if FPSW is live for a comparison result. They also use FPCW for exception masking control. Though the only exception they can generate is stack overflow and we manage the stack ourselves so that's not really going to occur.

    Reviewers: RKSimon, spatel

    Reviewed By: RKSimon

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D57934 (detail/ViewSVN)
    by ctopper
  7. [clang-tidy] Add options to bugprone-argument-comment to add missing argument comments to literals

    bugprone-argument-comment only supports identifying those comments which do not match the function parameter name

    This revision add 3 options to adding missing argument comments to literals (granularity on type is added to control verbosity of fixit)

    ```
    CheckOptions:
      - key:             bugprone-argument-comment.CommentBoolLiterals
        value:           '1'
      - key:             bugprone-argument-comment.CommentFloatLiterals
        value:           '1'
      - key:             bugprone-argument-comment.CommentIntegerLiterals
        value:           '1'
      - key:             bugprone-argument-comment.CommentStringLiterals
        value:           '1'
      - key:             bugprone-argument-comment.CommentCharacterLiterals
        value:           '1'
      - key:             bugprone-argument-comment.CommentUserDefinedLiterals
        value:           '1'
      - key:             bugprone-argument-comment.CommentNullPtrs
        value:           '1'
    ```

    After applying these options, literal arguments will be preceded with /*ParameterName=*/

    Reviewers: JonasToth, Eugene.Zelenko, alexfh, hokein, aaron.ballman

    Reviewed By: aaron.ballman, Eugene.Zelenko

    Differential Revision: https://reviews.llvm.org/D57674 (detail/ViewSVN)
    by paulhoad
  8. [X86] Add basic funnel shift demanded bits tests (detail/ViewSVN)
    by rksimon
  9. [x86] fix formatting; NFC

    (test commit #2 migrating to git) (detail/ViewSVN)
    by spatel
  10. [ASTMatchers][NFC] Update comments on assorted `CXXMemberCallExpr` matchers.

    Specifically:

    * fixes the comments on `hasObjectExpression`,
    * clarifies comments on `thisPointerType` and `on`,
    * adds comments to `onImplicitObjectArgument`.

    It also updates associated reference docs (using the doc tool).

    Reviewers: alexfh, steveire, aaron.ballman

    Differential Revision: https://reviews.llvm.org/D56849 (detail/ViewSVN)
    by ymandel
  11. [Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow

    Summary:
    Currently, `UnwrappedLineParser` thinks an arrow token after
    an ObjC method expression is a C++ lambda arrow, so it formats:

    ```
    [foo bar]->baz
    ```

    as:

    ```
    [foo bar] -> baz
    ```

    Because `UnwrappedLineParser` runs before `TokenAnnotator`, it can't
    know if the arrow token is after an ObjC method expression or not.

    This diff makes `TokenAnnotator` remove the TT_LambdaArrow on
    the arrow token if it follows an ObjC method expression.

    Test Plan: New test added. Ran test with:
      % ninja FormatTests && ./tools/clang/unittests/Format/FormatTests
      Confirmed test failed before diff and passed after diff.

    Reviewers: krasimir, djasper, sammccall

    Reviewed By: sammccall

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D57923 (detail/ViewSVN)
    by benhamilton
  12. [AMDGPU] Fix CS scratch setup on pre-GCN3 ASICs

    Summary:
    Prior to GCN3 s_load_dword offsets are in dwords rather than bytes.
    Thus the scratch buffer descriptor offset must be adjusted for pre-GCN3 ASICs.

    Reviewers: nhaehnle, tpr

    Reviewed By: nhaehnle

    Subscribers: sheredom, arsenm, kzhuravl, jvesely, wdng, yaxunl, dstuttard, t-tye, jfb, llvm-commits

    Differential Revision: https://reviews.llvm.org/D56496 (detail/ViewSVN)
    by critson
  13. [opaque pointer types] Cleanup CGBuilder's Create*GEP.

    Some of these functions take some extraneous arguments, e.g. EltSize,
    Offset, which are computable from the Type and DataLayout.

    Add some asserts to ensure that the computed values are consistent
    with the passed-in values, in preparation for eliminating the
    extraneous arguments. This also asserts that the Type is an Array for
    the calls named "Array" and a Struct for the calls named "Struct".

    Then, correct a couple of errors:

    1. Using CreateStructGEP on an array type. (this causes the majority
       of the test differences, as struct GEPs are created with i32
       indices, while array GEPs are created with i64 indices)

    2. Passing the wrong Offset to CreateStructGEP in TargetInfo.cpp on
       x86-64 NACL (which uses 32-bit pointers).

    Differential Revision: https://reviews.llvm.org/D57766 (detail/ViewSVN)
    by jyknight
  14. [scudo][standalone] Correct cmake copy/paste error

    Summary:
    I mistakenly used a SCUDO variable instead of SCUDO_STANDALONE one. The net
    result was that there were more architecture supported than I intended.
    Correct that, I'll add more architectures once the bots are enabled and
    green for x86.

    Reviewers: eugenis, vitalybuka

    Reviewed By: vitalybuka

    Subscribers: mgorny, delcypher, #sanitizers, llvm-commits

    Tags: #llvm, #sanitizers

    Differential Revision: https://reviews.llvm.org/D57897 (detail/ViewSVN)
    by cryptoad
  15. Revert r353416 "[DAG] Cleanup unused nodes on failed store-to-load forward combine."

    This cleanup causes out-of-tree crashes. (detail/ViewSVN)
    by niravd
  16. AMDGPU/GlobalISel: Fix shift legalization for non-power-of-2

    clampScalar doesn't do anything for non-power-of-2 in range.
    There should probably be a combination rule to reduce the number
    of matching rules. (detail/ViewSVN)
    by arsenm
  17. [clang-tidy][NFC] Fix typo.

    Fix placement of comma from previous (test) commit. (detail/ViewSVN)
    by ymandel
  18. [AMDGPU][MC] Added support of lds_direct operand

    See bug 39293: https://bugs.llvm.org/show_bug.cgi?id=39293

    Reviewers: artem.tamazov, rampitec

    Differential Revision: https://reviews.llvm.org/D57889 (detail/ViewSVN)
    by dpreobra
  19. [clang-tidy][NFC] Test commit. Add missing comma. (detail/ViewSVN)
    by ymandel
  20. AMDGPU/GlobalISel: Fix non-power-of-2 implicit_def (detail/ViewSVN)
    by arsenm
  21. [llvm-objcopy] Add few file processing directives

    Differential revision: https://reviews.llvm.org/D57877 (detail/ViewSVN)
    by evgeny777
  22. [MIPS GlobalISel] Select any extending load and truncating store

    Make behavior of G_LOAD in widenScalar same as for G_ZEXTLOAD and
    G_SEXTLOAD. That is perform widenScalarDst to size given by the target
    and avoid additional checks in common code. Targets can reorder or add
    additional rules in LegalizeRuleSet for the opcode to achieve desired
    behavior.

    Select extending load that does not have specified type of extension
    into zero extending load.

    Select truncating store that stores number of bytes indicated by size
    in MachineMemoperand.

    Differential Revision: https://reviews.llvm.org/D57454 (detail/ViewSVN)
    by petar.avramovic
  23. gn build: Merge r353471, r353373. (detail/ViewSVN)
    by nico
  24. AMDGPU/GlobalISel: Don't use a copy in addrspacecast lowering (detail/ViewSVN)
    by arsenm
  25. [AMDGPU][MC][CODEOBJECT] Added predefined symbols to access GPU minor and stepping numbers

    Added the following Code Object v3 symbols:
        .amdgcn.gfx_generation_minor
        .amdgcn.gfx_generation_stepping

    Reviewers: artem.tamazov, kzhuravl

    Differential Revision: https://reviews.llvm.org/D57826 (detail/ViewSVN)
    by dpreobra
  26. [clangd] Fix an assertion in TypoCorrection.

    Summary: https://github.com/clangd/clangd/issues/7

    Reviewers: sammccall, hokein

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

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D57944 (detail/ViewSVN)
    by ioeric
  27. [AMDGPU] Fix DPP combiner

    Differential revision: https://reviews.llvm.org/D55444

    dpp move with uses and old reg initializer should be in the same BB.
    bound_ctrl:0 is only considered when bank_mask and row_mask are fully enabled (0xF). Otherwise the old register value is checked for identity.
    Added add, subrev, and, or instructions to the old folding function.
    Kill flag is cleared for the src0 (DPP register) as it may be copied into more than one user.

    The pass is still disabled by default. (detail/ViewSVN)
    by vpykhtin
  28. [DWARF] LLVM ERROR: Broken function found, while removing Debug Intrinsics.

    Check that when SimplifyCFG is flattening a 'br', all their debug intrinsic instructions are removed, including any dbg.label referencing a label associated with the basic blocks being removed.

    Differential Revision: https://reviews.llvm.org/D57444 (detail/ViewSVN)
    by carlos.alberto.enciso
  29. Attempt to fix build bot after r353509 (detail/ViewSVN)
    by evgeny777
  30. [llvm-objcopy] Add --redefine-syms

    Differential revision: https://reviews.llvm.org/D57738 (detail/ViewSVN)
    by evgeny777
  31. Revert r353424 "[llvm-ar][libObject] Fix relative paths when nesting thin archives."

    This broke the Chromium build on Windows, see https://crbug.com/930058

    > Summary:
    > When adding one thin archive to another, we currently chop off the relative path to the flattened members. For instance, when adding `foo/child.a` (which contains `x.txt`) to `parent.a`, whe
    > lattening it we should add it as `foo/x.txt` (which exists) instead of `x.txt` (which does not exist).
    >
    > As a note, this also undoes the `IsNew` parameter of handling relative paths in r288280. The unit test there still passes.
    >
    > This was reported as part of testing the kernel build with llvm-ar: https://patchwork.kernel.org/patch/10767545/ (see the second point).
    >
    > Reviewers: mstorsjo, pcc, ruiu, davide, david2050
    >
    > Subscribers: hiraditya, llvm-commits
    >
    > Tags: #llvm
    >
    > Differential Revision: https://reviews.llvm.org/D57842

    This reverts commit bf990ab5aab03aa0aac53c9ef47ef264307804ed. (detail/ViewSVN)
    by hans
  32. [MIPS GlobalISel] Select mul

    Legalize and select G_MUL for s32 and smaller types for MIPS32.

    Differential Revision: https://reviews.llvm.org/D57816 (detail/ViewSVN)
    by petar.avramovic

Started by upstream project Clang Stage 2: cmake, R -g Asan&UBSan, using Stage 1 RA, Phase 1 build number 6119
originally caused by:

This run spent:

  • 1 min 56 sec waiting;
  • 24 min build duration;
  • 26 min total from scheduled to completion.

Identified problems

Missing test results

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

CMake Error

This build failed because of an CMake configuration error. Below is a list of all errors in the build log:
Indication 2