Started 2 yr 7 mo ago
Took 2 min 13 sec on green-dragon-03

Failed Build r302527 (#5801) (May 9, 2017 7:14:41 AM)

Subproject Builds

Revision: 302527
Changes
  1. Add extra operand to CALLSEQ_START to keep frame part set up previously

    Using arguments with attribute inalloca creates problems for verification
    of machine representation. This attribute instructs the backend that the
    argument is prepared in stack prior to  CALLSEQ_START..CALLSEQ_END
    sequence (see http://llvm.org/docs/InAlloca.htm for details). Frame size
    stored in CALLSEQ_START in this case does not count the size of this
    argument. However CALLSEQ_END still keeps total frame size, as caller can
    be responsible for cleanup of entire frame. So CALLSEQ_START and
    CALLSEQ_END keep different frame size and the difference is treated by
    MachineVerifier as stack error. Currently there is no way to distinguish
    this case from actual errors.

    This patch adds additional argument to CALLSEQ_START and its
    target-specific counterparts to keep size of stack that is set up prior to
    the call frame sequence. This argument allows MachineVerifier to calculate
    actual frame size associated with frame setup instruction and correctly
    process the case of inalloca arguments.

    The changes made by the patch are:
    - Frame setup instructions get the second mandatory argument. It
      affects all targets that use frame pseudo instructions and touched many
      files although the changes are uniform.
    - Access to frame properties are implemented using special instructions
      rather than calls getOperand(N).getImm(). For X86 and ARM such
      replacement was made previously.
    - Changes that reflect appearance of additional argument of frame setup
      instruction. These involve proper instruction initialization and
      methods that access instruction arguments.
    - MachineVerifier retrieves frame size using method, which reports sum of
      frame parts initialized inside frame instruction pair and outside it.

    The patch implements approach proposed by Quentin Colombet in
    https://bugs.llvm.org/show_bug.cgi?id=27481#c1.
    It fixes 9 tests failed with machine verifier enabled and listed
    in PR27481.

    Differential Revision: https://reviews.llvm.org/D32394 (detail/ViewSVN)
    by sepavloff

Started by upstream project phase2_modules_relay build number 3970
originally caused by:

This run spent:

  • 60 ms waiting;
  • 2 min 13 sec build duration;
  • 2 min 13 sec total from scheduled to completion.

Identified problems

Subproject Failed

This job failed because one of the jobs triggered by it failed.
Indication 1