Started 8 mo 13 days ago
Took 1 hr 48 min on green-dragon-19

Success Build rL:372878 - C:372884 - #776 (Sep 25, 2019 7:45:27 AM)

  • : 372878
  • : 372884
  • : 372849
  • : 364589
  • : 372433
  • : 372832
  • : 372875
  1. [NFC] Fix typo in `getPreviousDecl` comment. (detail/ViewSVN)
    by ymandel
  2. Revert r372863: [AST] Extract Decl::printNestedNameSpecifier helper from Decl::printQualifiedName

    Reason: causes a test failure, will investigate and re-land with a fix. (detail/ViewSVN)
    by ibiryukov
  3. [IR] allow fast-math-flags on phi of FP values (2nd try)

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

    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
  4. Fix analyzer TypeAttributeImpl::anchor() override.

    TypeAttributeImpl inherits from EnumAttributeImpl which already defines anchor() as a virtual, so we should override this instead of redeclaring it. (detail/ViewSVN)
    by rksimon
  5. [gn build] Fix Python DeprecationWarning

    This fixes two issues:
    - DeprecationWarning: invalid escape sequence \`
    - ResourceWarning: unclosed file

    Subscribers: mgorny, llvm-commits

    Tags: #llvm

    Differential Revision: (detail/ViewSVN)
    by mantognini
  6. [clangd] Fix parseNamespaceEvents to parse the last token

    Reviewers: hokein, ilya-biryukov

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

    Tags: #clang

    Differential Revision: (detail/ViewSVN)
    by kadircet
  7. [Dominators][AMDGPU] Don't use virtual exit node in findNearestCommonDominator. Cleanup MachinePostDominators.

    This patch fixes a bug that originated from passing a virtual exit block (nullptr) to `MachinePostDominatorTee::findNearestCommonDominator` and resulted in assertion failures inside its callee. It also applies a small cleanup to the class.

    The patch introduces a new function in PDT that given a list of `MachineBasicBlock`s finds their NCD. The new overload of `findNearestCommonDominator` handles virtual root correctly.

    Note that similar handling of virtual root nodes is not necessary in (forward) `DominatorTree`s, as right now they don't use virtual roots.

    Reviewers: tstellar, tpr, nhaehnle, arsenm, NutshellySima, grosser, hliao

    Reviewed By: hliao

    Subscribers: hliao, kzhuravl, jvesely, wdng, yaxunl, dstuttard, t-tye, hiraditya, llvm-commits

    Tags: #amdgpu, #llvm

    Differential Revision: (detail/ViewSVN)
    by kuhar
  8. [SystemZ]  Improve emitSelect()

    Merge more Select pseudo instructions in emitSelect() by allowing other
    instructions between them as long as they do not clobber CC.

    Debug value instructions are now moved down to below the new PHIs instead of
    erasing them.

    Review: Ulrich Weigand (detail/ViewSVN)
    by jonpa
  9. Fix cppcheck variable shadow warning. NFCI. (detail/ViewSVN)
    by rksimon
  10. Add missing ExplicitTy default initialization to ConstantExprKeyType constructors.

    Fixes cppcheck uninitialized variable warnings (detail/ViewSVN)
    by rksimon

Started by an SCM change (8 times)

This run spent:

  • 1 hr 3 min waiting;
  • 1 hr 48 min build duration;
  • 2 hr 52 min total from scheduled to completion.
LLVM/Clang Warnings: 1 warning.
    Test Result (no failures)