Started 8 mo 21 days ago
Took 1 hr 37 min on green-dragon-22

Success Build rL:371780 - C:371781 - #568 (Sep 12, 2019 3:15:46 PM)

  • : 371780
  • : 371781
  • : 371758
  • : 364589
  • : 371690
  • : 371763
  • : 371773
  1. [analyzer] Fix the 'analyzer-enabled-checkers.c' test on non-linux machines.

    '-Xclang -triple' doesn't seem to override the default target triple
    as reliably as '--target'. This leads to test failing due to
    platform-specific checks getting unexpectedly enabled. (detail/ViewSVN)
    by dergachev
  2. [AArch64][GlobalISel] Support sibling calls with outgoing arguments

    This adds support for lowering sibling calls with outgoing arguments.


    define void @foo(i32 %a)

    Support is ported from AArch64ISelLowering's `isEligibleForTailCallOptimization`.
    The only thing that is missing is a full port of
    `TargetLowering::parametersInCSRMatch`. So, if we're using swiftself,
    we'll never tail call.

    - Rename `analyzeCallResult` to `analyzeArgInfo`, since the function is now used
      for both outgoing and incoming arguments
    - Teach `OutgoingArgHandler` about tail calls. Tail calls use frame indices for
      stack arguments.
    - Teach `lowerFormalArguments` to set the bytes in the caller's stack argument
      area. This is used later to check if the tail call's parameters will fit on
      the caller's stack.
    - Add `areCalleeOutgoingArgsTailCallable` to perform the eligibility check on
      the callee's outgoing arguments.

    For testing:

    - Update call-translator-tail-call to verify that we can now tail call with
      outgoing arguments, use G_FRAME_INDEX for stack arguments, and respect the
      size of the caller's stack
    - Remove GISel-specific check lines from speculation-hardening.ll, since GISel
      now tail calls like the other selectors
    - Add a GISel test line to tailcall-string-rvo.ll since we can tail call in that
      test now
    - Add a GISel test line to tailcall_misched_graph.ll since we tail call there
      now. Add specific check lines for GISel, since the debug output from the
      machine-scheduler differs with GlobalISel. The dependency still holds, but
      the output comes out in a different order.

    Differential Revision: (detail/ViewSVN)
    by paquette
  3. [PowerPC] Remove the SPE4RC register class and instead add f32 to the GPRC register class.

    Since the SPE4RC register class contains an identical set of registers
    and an identical spill size to the GPRC class its slightly confusing
    the tablegen emitter. It's preventing the GPRC_and_GPRC_NOR0 synthesized
    register class from inheriting VTs and AltOrders from GPRC or GPRC_NOR0.
    This is because SPE4C is found first in the super register class list
    when inheriting these properties and it doesn't set the VTs or
    AltOrders the same way as GPRC or GPRC_NOR0.

    This patch replaces all uses of GPE4RC with GPRC and allows GPRC and
    GPRC_NOR0 to contain f32.

    The test changes here are because the AltOrders are being inherited
    to GPRC_NOR0 now.

    Found while trying to determine if getCommonSubClass needs to take
    a VT argument. It was originally added to support fp128 on x86-64,
    I've changed some things about that so that it might be needed
    anymore. But a PowerPC test crashed without it and I think its
    due to this subclass issue.

    Reviewers: jhibbits, nemanjai, kbarton, hfinkel

    Subscribers: wuzish, nemanjai, mehdi_amini, hiraditya, kbarton, MaskRay, dexonsmith, jsji, shchenz, steven.zhang, llvm-commits

    Tags: #llvm

    Differential Revision: (detail/ViewSVN)
    by ctopper
  4. Remove a duplicate test

    Turns out I'd already added exactly the same test under the name non_unit_stride. (detail/ViewSVN)
    by reames
  5. [SCEV] Add smin support to getRangeRef

    We were failing to compute trip counts (both exact and maximum) for any loop which involved a comparison against either an umin or smin. It looks like this simply got missed when we added smin/umin to SCEV.  (Note: umin was submitted separately earlier today.  Turned out two folks hit this at the same time.)

    Differential Revision: (detail/ViewSVN)
    by reames

Started by an SCM change (3 times)

This run spent:

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