Started 1 mo 2 days ago
Took 17 hr on green-dragon-08

Failed Build #5479 (Sep 16, 2019 7:08:08 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 371989
  • http://llvm.org/svn/llvm-project/cfe/trunk : 371985
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 371941
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 371925
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 371986
Changes
  1. [X86][NFC] Add a `use-aa` feature.

    Summary:
    This allows enabling useaa on the command-line and will allow enabling the
    feature on a per-CPU basis where benchmarking shows improvements.

    This is modelled after the ARM/AArch64 target.

    Reviewers: RKSimon, andreadb, craig.topper

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

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D67266 (detail/ViewSVN)
    by courbet
  2. [InstCombine] add/move tests for icmp with add operand; NFC (detail/ViewSVN)
    by spatel
  3. [clangd][vscode] update the development doc. (detail/ViewSVN)
    by hokein
  4. Move some definitions from Sema to Basic to fix shared libs build

    r371875 moved some functionality around to a Basic header file, but
    didn't move its definitions as well.  This patch moves some things
    around so that shared library building can work. (detail/ViewSVN)
    by erichkeane
  5. [docs][llvm-strings] Write llvm-strings documentation

    Previously we only had a stub document.

    Reviewed by: MaskRay

    Differential Revision: https://reviews.llvm.org/D67554 (detail/ViewSVN)
    by jhenderson
  6. [docs][llvm-size] Write llvm-size documentation

    Previously we only had a stub document.

    Reviewed by: serge-sans-paille, MaskRay

    Differential Revision: https://reviews.llvm.org/D67555 (detail/ViewSVN)
    by jhenderson
  7. [ARM] Fold VCMP into VPT

    MVE has VPT instructions, which perform the duties of both a VCMP and a VPST in
    a single instruction, performing the compare and starting the VPT block in one.
    This teaches the MVEVPTBlockPass to fold them, searching back through the
    basicblock for a valid VCMP and creating the VPT from its operands.

    There are some changes to the VPT instructions to accommodate this, altering
    the order of the operands to match the VCMP better, and changing P0 register
    defs to be VPR defs, as is used in other places.

    Differential Revision: https://reviews.llvm.org/D66577 (detail/ViewSVN)
    by dmgreen
  8. [InstCombine] remove unneeded one-use checks for icmp fold

    This fold and several others were added in:
    rL125734 <https://reviews.llvm.org/rL125734>
    ...with no explanation for the one-use checks other than the code
    comments about register pressure.

    Given that this is IR canonicalization, we shouldn't be worried
    about register pressure though; the backend should be able to
    adjust for that as needed.

    This is part of solving PR43310 the theoretically right way:
    https://bugs.llvm.org/show_bug.cgi?id=43310
    ...ie, if we don't cripple basic transforms, then we won't
    need to add special-case code to detect larger patterns.

    rL371940 is a related patch in this series. (detail/ViewSVN)
    by spatel
  9. [clangd] Bump vscode-clangd v0.0.17

    CHANGELOG:
    - added semantic highlighting support (under the clangd.semanticHighlighting
      flag);
    - better error message when clangd fails to execute refactoring-like
      actions;
    - improved the readme doc; (detail/ViewSVN)
    by hokein
  10. [InstCombine] add icmp tests with extra uses; NFC (detail/ViewSVN)
    by spatel
  11. [InstCombine] fix comments to match code; NFC

    This blob was written before match() existed, so it
    could probably be reduced significantly.

    But I suspect it isn't well tested, so tests would have
    to be added to reduce risk from logic changes. (detail/ViewSVN)
    by spatel
  12. gn build: Merge r371976 (detail/ViewSVN)
    by nico
  13. Implement semantic selections.

    Summary:
    For a given cursor position, it returns ranges that are interesting to the user.
    Currently the semantic ranges correspond to the nodes of the syntax trees.

    Subscribers: mgorny, jkorous, arphaman, kadircet, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D67358 (detail/ViewSVN)
    by usaxena95
  14. [VPlanSLP] Don't dereference a cast_or_null<VPInstruction> result. NFCI.

    The static analyzer is warning about a potential null dereference of the cast_or_null result, I've split the cast_or_null check from the ->getUnderlyingInstr() call to avoid this, but it appears that we weren't seeing any null pointers in the dumped bundles in the first place. (detail/ViewSVN)
    by rksimon
  15. [SLPVectorizer] Assert that we find a LastInst to silence analyzer null dereference warning. NFCI. (detail/ViewSVN)
    by rksimon
  16. [SLPVectorizer] Don't dereference a dyn_cast result. NFCI.

    The static analyzer is warning about potential null dereferences of dyn_cast<> results - in these cases we can safely use cast<> directly as we know that these cases should all be the correct type, which is why its working atm and anyway cast<> will assert if they aren't. (detail/ViewSVN)
    by rksimon
  17. Added return statement to fix compile and build warning:

    llvm-rtdyld.cpp:966:7: warning: variable ‘Result’ set but not used (detail/ViewSVN)
    by sjoerdmeijer
  18. [clangd] Fix a crash when renaming operator.

    Summary:
    The renamelib uses a tricky way to calculate the end location by relying
    on decl name, this is incorrect for the overloaded operator (the name is
    "operator++" instead of "++"), which will cause out-of-file offset.

    We also disable renaming operator symbol, this case is tricky, and
    renamelib doesnt handle it properly.

    Reviewers: ilya-biryukov

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

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D67607 (detail/ViewSVN)
    by hokein
  19. Change signature of __builtin_rotateright64 back to unsigned

    The signature of __builtin_rotateright64 was by misstake changed from
    unsigned to signed in r360863, this patch will change it back to
    unsigned as intended.

    This fixes pr43309

    Reviewers: efriedma, hans

    Reviewed By: hans

    Differential Revision: https://reviews.llvm.org/D67606 (detail/ViewSVN)
    by karka
  20. Fix the rst doc, unbreak buildbot. (detail/ViewSVN)
    by hokein
  21. [SVE][Inline-Asm] Add constraints for SVE predicate registers

    Summary:
    Adds the following inline asm constraints for SVE:
      - Upl: One of the low eight SVE predicate registers, P0 to P7 inclusive
      - Upa: SVE predicate register with full range, P0 to P15

    Reviewers: t.p.northover, sdesmalen, rovka, momchil.velikov, cameron.mcinally, greened, rengolin

    Reviewed By: rovka

    Subscribers: javed.absar, tschuett, rkruppe, psnobl, cfe-commits, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D66524 (detail/ViewSVN)
    by kmclaughlin
  22. gn build: Merge r371965 (detail/ViewSVN)
    by nico
  23. [clang-tidy] performance-inefficient-vector-operation: Support proto repeated field

    Summary:
    Finds calls that add element to protobuf repeated field in a loop
    without calling Reserve() before the loop. Calling Reserve() first can avoid
    unnecessary memory reallocations.

    A new option EnableProto is added to guard this feature.

    Patch by Cong Liu!

    Reviewers: gribozavr, alexfh, hokein, aaron.ballman

    Reviewed By: hokein

    Subscribers: lebedev.ri, xazax.hun, Eugene.Zelenko, cfe-commits

    Tags: #clang, #clang-tools-extra

    Differential Revision: https://reviews.llvm.org/D67135 (detail/ViewSVN)
    by hokein
  24. gn build: Merge r371959 (detail/ViewSVN)
    by nico
  25. [AArch64] Some more FP16 FMA pattern matching

    After our previous machinecombiner exercises (rL371321, rL371818, rL371833), we
    were still missing a few FP16 FMA patterns.

    Differential Revision: https://reviews.llvm.org/D67576 (detail/ViewSVN)
    by sjoerdmeijer
  26. [SystemZ]  Merge the SystemZExpandPseudo pass into SystemZPostRewrite.

    SystemZExpandPseudo:s only job was to expand LOCRMux instructions into jump
    sequences. This needs to be done if expandLOCRPseudo() or expandSELRPseudo()
    fails to find a legal opcode (all registers "high" or "low"). This task has
    now been moved to SystemZPostRewrite while removing the SystemZExpandPseudo
    pass.

    It is in fact preferred to expand these pseudos directly after register
    allocation in SystemZPostRewrite since the hinted register combinations are
    then not subject to later optimizations.

    Review: Ulrich Weigand
    https://reviews.llvm.org/D67432 (detail/ViewSVN)
    by jonpa
  27. AMDGPU/GlobalISel: Remove illegal select tests

    These fail in a release build. (detail/ViewSVN)
    by arsenm
  28. AMDGPU/GlobalISel: Select SMRD loads for more types (detail/ViewSVN)
    by arsenm
  29. AMDGPU/GlobalISel: RegBankSelect for kill (detail/ViewSVN)
    by arsenm
  30. AMDGPU/GlobalISel: Legalize s1 source G_[SU]ITOFP (detail/ViewSVN)
    by arsenm
  31. AMDGPU/GlobalISel: Set type on vgpr live in special arguments

    Fixes assertion with workitem ID intrinsics used in non-kernel
    functions. (detail/ViewSVN)
    by arsenm
  32. AMDGPU/GlobalISel: Select S16->S32 fptoint (detail/ViewSVN)
    by arsenm
  33. AMDGPU/GlobalISel: Select s32->s16 G_[US]ITOFP (detail/ViewSVN)
    by arsenm
  34. AMDGPU/GlobalISel: Fix VALU s16 fneg (detail/ViewSVN)
    by arsenm
  35. [Attributor] Heap-To-Stack Conversion

    D53362 gives a prototype heap-to-stack conversion pass. With addition of new attributes in the attributor, this can now be revisted and improved. This will place it in the Attributor to make it easier to use new attributes (eg. nofree, nosync, willreturn, etc.) and other attributor features.

    Reviewers: jdoerfert, uenoku, hfinkel, efriedma

    Subscribers: lebedev.ri, xbolva00, hiraditya, llvm-commits

    Differential Revision: https://reviews.llvm.org/D65408 (detail/ViewSVN)
    by sstefan
  36. Commit missing part of "Split many_tls_keys.cpp into two tests"

    https://reviews.llvm.org/D67428

    This change was lost due to a file rename and modification. (detail/ViewSVN)
    by kamil
  37. [InstCombine] remove unneeded one-use checks for icmp fold

    This fold and several others were added in:
    rL125734
    ...with no explanation for the one-use checks other than the code
    comments about register pressure.

    Given that this is IR canonicalization, we shouldn't be worried
    about register pressure though; the backend should be able to
    adjust for that as needed.

    There are similar checks as noted with the TODO comments. I'm
    hoping to remove those restrictions too, but if any of these
    does cause a regression, it should be easier to correct by making
    small, individual commits.

    This is part of solving PR43310 the theoretically right way:
    https://bugs.llvm.org/show_bug.cgi?id=43310
    ...ie, if we don't cripple basic transforms, then we won't
    need to add special-case code to detect larger patterns. (detail/ViewSVN)
    by spatel
  38. [InstCombine] add icmp tests with extra uses; NFC (detail/ViewSVN)
    by spatel
  39. [PowerPC][NFC] Add a testcase for fdiv expansion.

    Pre-commit for following patch. (detail/ViewSVN)
    by jsji

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18328
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18329
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18330
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18331
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18332
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18333
originally caused by:

This run spent:

  • 10 hr waiting;
  • 17 hr build duration;
  • 1 day 3 hr total from scheduled to completion.

Identified problems

Compile Error

This build failed because of a compile error. Below is a list of all errors in the build log:
Indication 1

Missing test results

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

Regression test failed

This build failed because a regression test in the test suite FAILed. See the test report for details.
Indication 3

Ninja target failed

Below is a link to the first failed ninja target.
Indication 4