Started 2 yr 8 mo ago
Took 1 hr 28 min on green-dragon-13

Success Build r302457 (#5774) (May 8, 2017 1:46:48 PM)

Subproject Builds

Revision: 302457
  1. Set ENABLE_WERROR=OFF for ABITestsuitBuilder. (detail/ViewSVN)
    by gkistanova
  2. Add a blurb to the release notes about the WeakVH -> WeakTrackingVH transition (detail/ViewSVN)
    by sanjoy
  3. Fixed 'haltOnFailure' for build step. (detail/ViewSVN)
    by gkistanova
  4. [CodeView] Add support for random access type visitors.

    Previously type visitation was done strictly sequentially, and
    TypeIndexes were computed by incrementing the TypeIndex of the
    last visited record.  This works fine for situations like dumping,
    but not when you want to visit types in random order.  For example,
    in a debug session someone might lookup a symbol by name, find that
    it has TypeIndex 10,000 and then want to go straight to TypeIndex

    In order to make this work, the visitation framework needs a mode
    where it can plumb TypeIndices through the callback pipeline.  This
    patch adds such a mode.  In doing so, it is necessary to provide
    an alternative implementation of TypeDatabase that supports random
    access, so that is done as well.

    Nothing actually uses these random access capabilities yet, but
    this will be done in subsequent patches.

    Differential Revision: (detail/ViewSVN)
    by zturner
  5. [AArch64][RegisterBankInfo] Change the default mapping of fp loads.

    This fixes PR32550, in a way that does not imply running the greedy
    mode at O0.

    The fix consists in checking if a load is used by any floating point
    instruction and if yes, we return a default mapping with FPR instead
    of GPR. (detail/ViewSVN)
    by qcolombet
  6. [AArch64][RegisterBankInfo] Fix mapping cost for GPR.

    In r292478, we changed the order of the enum that is referenced by
    PMI_FirstXXX. This had the side effect of changing the cost of the
    mapping of all the loads, instead of just the FPRs ones.

    Reinstate the higher cost for all but GPR loads.
    Note: This did not have any external visible effects:
    - For Fast mode, the cost would have been higher, but we don't care
      because we don't try to use alternative mappings.
    - For Greedy mode, the higher cost of the GPR loads, would have
      triggered the use of the supposedly alternative mapping, that
      would be in fact the same GPR mapping but with a lower cost. (detail/ViewSVN)
    by qcolombet
  7. [InstCombine] move/add tests for not(shr (not X), Y); NFC (detail/ViewSVN)
    by spatel
  8. [ARM] Use a Changed flag to avoid making a pass's return value dependent on a compare with a Statistic object.

    Statistic compile to always be 0 in release build so this compare would always return false. And in the debug builds Statistic are global variables and remember their values across pass runs. So this compare returns true anytime the pass runs after the first time it modifies something.

    This was found after reviewing all usages of comparison operators on a Statistic object. We had some internal code that did a compare with a statistic that caused a mismatch in output between debug and release builds. So we did an audit out of paranoia. (detail/ViewSVN)
    by ctopper

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

This run spent:

  • 7 ms waiting;
  • 1 hr 28 min build duration;
  • 1 hr 28 min total from scheduled to completion.