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

Failed Build #292 (Feb 14, 2019 1:39:54 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 354062
  • http://llvm.org/svn/llvm-project/cfe/trunk : 354063
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 354064
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 346271
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 353955
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 353926
Changes
  1. Set hidden attribute on lprofMergeValueProfData

    Summary:
    The changes in https://reviews.llvm.org/D44847 cause load time failure
    due to lprofMergeValueProfData in Android libs enabled with profile generation:

    "dlopen failed: cannot locate symbol "lprofMergeValueProfData" referenced by..."

    Marking lprofMergeValueProfData as hidden so the correct in-module definition
    is picked by the linker.

    Reviewers: davidxl

    Reviewed By: davidxl

    Subscribers: efriedma, xur, davidxl, llvm-commits

    Differential Revision: https://reviews.llvm.org/D55893 (detail/ViewSVN)
    by apazos
  2. Further relax restriction in tests to include where "-E" and "-S" must appear.

    Also updated a few instances of "-emit-llvm-bc" and "-emit-obj" that were missed in the previous change. (detail/ViewSVN)
    by dyung
  3. [ThinLTO] Detect partially split modules during the thin link

    Summary:
    The changes to disable LTO unit splitting by default (r350949) and
    detect inconsistently split LTO units (r350948) are causing some crashes
    when the inconsistency is detected in multiple threads simultaneously.
    Fix that by having the code always look for the inconsistently split
    LTO units during the thin link, by checking for the presence of type
    tests recorded in the summaries.

    Modify test added in r350948 to remove single threading required to fix
    a bot failure due to this issue (and some debugging options added in the
    process of diagnosing it).

    Reviewers: pcc

    Subscribers: mehdi_amini, inglorion, eraman, steven_wu, dexonsmith, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D57561 (detail/ViewSVN)
    by tejohnson
  4. [libFuzzer] better stats for the fork mode (detail/ViewSVN)
    by kcc
  5. [CMake] Fix ability to use LLVM_ENABLE_PROJECTS with LLVM_EXTERNAL_PROJECTS

    LLVM r353148, changed the circumstances in which the project source directory variables are created to only create them for LLVM projects. This patch initializes the directory variables for projects specified in `LLVM_EXTERNAL_PROJECTS` as well. (detail/ViewSVN)
    by cbieneman
  6. [InstCombine] Add todos for possible atomicrmw transforms (detail/ViewSVN)
    by reames
  7. Canonicalize all integer "idempotent" atomicrmw ops

    For "idempotent" atomicrmw instructions which we can't simply turn into load, canonicalize the operation and constant. This reduces the matching needed elsewhere in the optimizer, but doesn't directly impact codegen.

    For any architecture where OR/Zero is not a good default choice, you can extend the AtomicExpand lowerIdempotentRMWIntoFencedLoad mechanism. I reviewed X86 to make sure this works well, haven't audited other backends.

    Differential Revision: https://reviews.llvm.org/D58244 (detail/ViewSVN)
    by reames
  8. Stop enabling clang-tools-extra automatically when clang is in LLVM_ENABLE_PROJECTS

    If you want to build clang-tools-extra with monorepo, just add it to
    LLVM_ENABLE_PROJECTS like with other projects.

    See also "Separating clang-tools-extra from clang in LLVM_ENABLE_PROJECTS"
    on cfe-dev.

    Differential Revision: https://reviews.llvm.org/D58157 (detail/ViewSVN)
    by nico
  9. [CodeGenObjC] Emit [[X alloc] init] as objc_alloc_init(X) when available

    This provides a code size win on the caller side, since the init
    message send is done in the runtime function.

    rdar://44987038

    Differential revision: https://reviews.llvm.org/D57936 (detail/ViewSVN)
    by epilk
  10. Optional specialization for trivially copyable types, part2 (detail/ViewSVN)
    by serge_sans_paille
  11. [X86] Add clang support for X86 flag output parameters.

    Summary:
    Add frontend support and expected flags for X86 inline assembly flag
    parameters.

    Reviewers: craig.topper, rnk, echristo

    Subscribers: eraman, nickdesaulniers, void, llvm-commits

    Differential Revision: https://reviews.llvm.org/D57394 (detail/ViewSVN)
    by niravd
  12. Recommit Optional specialization for trivially copyable types

    Unfortunately the original code gets misscompiled by GCC (at least 8.1),
    this is a tentative workaround using std::memcpy instead of inplace new
    for trivially copyable types. I'll revert if it breaks.

    Original revision: https://reviews.llvm.org/D57097 (detail/ViewSVN)
    by serge_sans_paille
  13. Teach instcombine about remaining "idempotent" atomicrmw types

    Expand on Quentin's r353471 patch which converts some atomicrmws into loads. Handle remaining operation types, and fix a slight bug. Atomic loads are required to have alignment. Since this was within the InstCombine fixed point, somewhere else in InstCombine was adding alignment before the verifier saw it, but still, we should fix.

    Terminology wise, I'm using the "idempotent" naming that is used for the same operations in AtomicExpand and X86ISelLoweringInfo. Once this lands, I'll add similar tests for AtomicExpand, and move the pattern match function to a common location. In the review, there was seemingly consensus that "idempotent" was slightly incorrect for this context.  Once we setle on a better name, I'll update all uses at once.

    Differential Revision: https://reviews.llvm.org/D58242 (detail/ViewSVN)
    by reames
  14. Support: use internal `call_once` on PPC64le

    Always use the internal `call_once` for PPC64le.  This is needed to
    support the Swift toolchain on PPC64le.

    Patch by Sarvesh Tamba! (detail/ViewSVN)
    by Saleem Abdulrasool
  15. [llvm-ar] Implement the P modifier.

    Summary:
    GNU ar has a `P` modifier that changes filename comparisons to use full paths instead of the basename. As noted in the GNU docs, regular archives are not created with full path names, so P is used to deal with archives created by other archive programs (e.g. see the updated `absolute-paths.test` test case).

    Since thin archives use full path names -- paths are relative to the archive -- it seems very error prone to not imply P when dealing with thin archives, so P is implied in those cases. (I think this is a deviation from GNU ar that makes sense).

    This fixes PR37436 via https://github.com/ClangBuiltLinux/linux/issues/33.

    Reviewers: mstorsjo, pcc, ruiu, davide, david2050, rnk

    Subscribers: tpimh, llvm-commits, nickdesaulniers

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D57927 (detail/ViewSVN)
    by rupprecht
  16. [X86] cleanup inline asm register generation. NFCI. (detail/ViewSVN)
    by niravd
  17. [SystemZ] Do not emit VEXTEND or VROUND nodes without vector support.

    Review: Ulrich Weigand
    https://reviews.llvm.org/D58240 (detail/ViewSVN)
    by jonpa
  18. [Tests] Add tests for all idemptotent atomicrmws

    Base for a followup patch to strengthen the InstCombine transform, and then integrate the ExpandAtomics logic. (detail/ViewSVN)
    by reames
  19. [Sema] Fix a regression introduced in "[AST][Sema] Remove CallExpr::setNumArgs"

    D54902 removed CallExpr::setNumArgs in preparation of tail-allocating the
    arguments of CallExpr. It did this by allocating storage for
    max(number of arguments, number of parameters in the prototype). The
    temporarily nulled arguments however causes issues in BuildResolvedCallExpr
    when typo correction is done just after the creation of the call expression.

    This was unfortunately missed by the tests /:

    To fix this, delay setting the number of arguments to
    max(number of arguments, number of parameters in the prototype) until we are
    ready for it. It would be nice to have this encapsulated in CallExpr but this
    is the best I can come up with under the constraint that we cannot add
    anything the CallExpr.

    Fixes PR40286.

    Differential Revision: https://reviews.llvm.org/D57948

    Reviewed By: aaron.ballman (detail/ViewSVN)
    by brunoricci
  20. [X86][AVX] Add PR40730 test case (detail/ViewSVN)
    by rksimon
  21. Refine ArgPromotion metadata handling

    Summary:
    In r353537 we now copy all metadata to the new function, with the old
    being removed when the old function is eliminated. In some cases the old
    function is dropped to a declaration (seems to only occur with the old
    PM). Go ahead and clear all metadata from the old function to handle that
    case, since verification will complain otherwise. This is consistent
    with what was being done for debug metadata before r353537.

    Reviewers: davidxl, uabelho

    Subscribers: jdoerfert, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D58215 (detail/ViewSVN)
    by tejohnson
  22. [LoopUnrollPeel] Add case where we should forget the peeled loop from SCEV.

    The test case requires the peeled loop to be forgotten after peeling,
    even though it does not have a parent. When called via the unroller,
    SE->forgetTopmostLoop is also called, so the test case would also pass
    without any SCEV invalidation, but peelLoop is exposed as utility
    function. Also, in the test case, simplifyLoop will make changes,
    removing the loop from SCEV, but it is better to not rely on this
    behavior.

    Reviewers: sanjoy, mkazantsev

    Reviewed By: mkazantsev

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D58192 (detail/ViewSVN)
    by fhahn
  23. [RISCV][NFC] Add RV64I CHECK lines to inline-asm.ll test (detail/ViewSVN)
    by asb
  24. [ASTImporter] Check visibility/linkage of functions and variables

    Summary:
    During import of a global variable with external visibility the lookup
    will find variables (with the same name) but with static visibility.
    Clearly, we cannot put them into the same redecl chain.  The same is
    true in case of functions.  In this fix we filter the lookup results and
    consider only those which have the same visibility as the decl we
    currently import.

    We consider two decls in two anonymous namsepaces to have the same
    visibility only if they are imported from the very same translation
    unit.

    Reviewers: a_sidorin, shafik, a.sidorin

    Reviewed By: shafik

    Subscribers: jdoerfert, balazske, rnkovacs, dkrupp, Szelethus, gamesh411, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D57232 (detail/ViewSVN)
    by martong
  25. Reapply [VFS] Allow multiple RealFileSystem instances with independent CWDs.

    This reverts commit r351091.
    The original mac breakages are addressed by ensuring the root directory
    we're working from is fully symlink-resolved before starting.

    Differential Revision: https://reviews.llvm.org/D58169 (detail/ViewSVN)
    by sammccall
  26. [MIPS GlobalISel] Select phi instruction for integers

    Select G_PHI for integers for MIPS32.

    Differential Revision: https://reviews.llvm.org/D58183 (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 6135
originally caused by:

This run spent:

  • 9.5 sec waiting;
  • 13 min build duration;
  • 13 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