Started 8 mo 3 days ago
Took 1 hr 48 min on green-dragon-20

Failed Build rL:372868 - C:372870 - #775 (Sep 25, 2019 6:37:33 AM)

  • : 372868
  • : 372870
  • : 372849
  • : 364589
  • : 372433
  • : 372832
  • : 372860
  1. [libTooling] Introduce the MatchConsumer abstraction

    This revision introduces a separate (small) library for the `MatchConsumer`
    abstraction: computations over AST match results.  This abstraction is central
    to the Transformer framework, and there deserves being defined explicitly.

    Reviewers: gribozavr

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: (detail/ViewSVN)
    by ymandel
  2. Revert [IR] allow fast-math-flags on phi of FP values

    This reverts r372866 (git commit dec03223a97af0e4dfcb23da55c0f7f8c9b62d00) (detail/ViewSVN)
    by spatel
  3. [llvm-readobj/llvm-readelf] - .stack_sizes: demangle symbol names in warnings reported.

    I started this patch as a refactoring, tried to make a helper for
    getting symbol names, similar to how we get section names
    used in warning messages.

    So this patch cleanups the code and fixes an issue: symbol names
    in warning messages were not demangled.

    Differential revision: (detail/ViewSVN)
    by grimar
  4. [IR] allow fast-math-flags on phi of FP values

    The changes here are based on the corresponding diffs for allowing FMF on 'select':

    As discussed there, we want to have fast-math-flags be a property of an FP value
    because the alternative (having them on things like fcmp) leads to logical
    inconsistency such as:

    The earlier patch for select made almost no practical difference because most
    unoptimized conditional code begins life as a phi (based on what I see in clang).
    Similarly, I don't expect this patch to do much on its own either because
    SimplifyCFG promptly drops the flags when converting to select on a minimal
    example like:

    But once we have this plumbing in place, we should be able to wire up the FMF
    propagation and start solving cases like that.

    The change to RecurrenceDescriptor::AddReductionVar() is required to prevent a
    regression in a LoopVectorize test. We are intersecting the FMF of any
    FPMathOperator there, so if a phi is not properly annotated, new math
    instructions may not be either. Once we fix the propagation in SimplifyCFG, it
    may be safe to remove that hack.

    Differential Revision: (detail/ViewSVN)
    by spatel
  5. [docs][llvm-strings] Clarify "printable character" wording

    The --bytes option uses the phrase "printable ASCII characters", but the
    description section used simply "printable characters". To avoid any
    confusion about locale impacts etc, this change adopts the former's
    phrasing in both places. It also fixes a minor grammar issue in the

    Reviewed by: MaskRay

    Differential Revision: (detail/ViewSVN)
    by jhenderson
  6. [docs][llvm-strip] Update llvm-strip doc to better match llvm-objcopy's

    Main changes are mostly wording of some options, but this change also
    fixes a switch reference so that a link is created and moves
    --strip-sections into the ELF-specific area since it is only supported
    for ELF currently. (detail/ViewSVN)
    by jhenderson
  7. [AST] Extract Decl::printNestedNameSpecifier helper from Decl::printQualifiedName

    To be used in clangd, e.g. in D66647.
    Currently the alternative to this function is doing string manipulation on results of `printQualifiedName`, which is
    hard-to-impossible to get right in presence of template arguments.

    Reviewers: kadircet, aaron.ballman

    Reviewed By: kadircet, aaron.ballman

    Subscribers: aaron.ballman, usaxena95, cfe-commits

    Tags: #clang

    Differential Revision: (detail/ViewSVN)
    by ibiryukov
  8. [clangd] Add missing header guard, NFC. (detail/ViewSVN)
    by hokein
  9. [clangd] A helper to find explicit references and their names

    Allows to simplify pending code tweaks:
      - the upcoming DefineInline tweak (D66647)
      - remove using declaration (D56612)
      - qualify name under cursor (D56610)

    Another potential future application is simplifying semantic highlighting.

    Reviewers: kadircet

    Reviewed By: kadircet

    Subscribers: mgrang, jfb, MaskRay, jkorous, arphaman, usaxena95, cfe-commits

    Tags: #clang

    Differential Revision: (detail/ViewSVN)
    by ibiryukov
  10. [AMDGPU][MC][DOC] Updated AMD GPU assembler description.

    Summary of changes:
    - Updated to reflect recent changes in assembler;
    - Minor bugfixing and improvements. (detail/ViewSVN)
    by dpreobra
  11. [DAG] Pull out minimum shift value calc into a helper function. NFCI. (detail/ViewSVN)
    by rksimon
  12. [llvm-readobj] - Simplify stack-sizes.test test case.

    This is a follow-up for D67757,
    which allows to describe .stack_sizes sections with a new
    YAML syntax.

    Differential revision: (detail/ViewSVN)
    by grimar
  13. [yaml2elf] - Support describing .stack_sizes sections using unique suffixes.

    Currently we can't use unique suffixes in section names to describe
    stack sizes sections. E.g. '.stack_sizes [1]' will be treated as a regular section.
    This happens because we recognize stack sizes section by name and
    do not yet drop the suffix before the check.

    The patch fixes it.

    Differential revision: (detail/ViewSVN)
    by grimar
  14. gn build: (manually) merge r372843 (detail/ViewSVN)
    by nico

Started by an SCM change (12 times)

This run spent:

  • 1 hr 40 min waiting;
  • 1 hr 48 min build duration;
  • 3 hr 29 min total from scheduled to completion.
LLVM/Clang Warnings: 2 warnings.
Test Result (1 failure / +1)

Identified problems

Ninja target failed

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

Regression test failed

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

Compile Error

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