Started 1 mo 3 days ago
Took 7 min 50 sec

Success Build clang-r366565-t58125-b58125.tar.gz (Jul 19, 2019 6:15:07 AM)

Issues

No known issues detected

Build Log

Revision: 366290
Changes
  1. [NFC][InstCombine] Tests for 'rem' formation from sub-of-mul-by-'div' (PR42673)

    https://rise4fun.com/Alive/8Rp
    https://bugs.llvm.org/show_bug.cgi?id=42673 (detail)
    by lebedevri
  2. [NFC][InstCombine] Redundant masking before left-shift: tests with assume

    If the legality check is `(shiftNbits-maskNbits) s>= 0`,
    then we can simplify it to `shiftNbits u>= maskNbits`,
    which is easier to check for.

    However, currently switching the `dropRedundantMaskingOfLeftShiftInput()`
    to `SimplifyICmpInst()` does not catch these cases and regresses
    currently-handled cases, so i'll leave it as is for now.

    https://rise4fun.com/Alive/25P (detail)
    by lebedevri
  3. Fix MSVC "result of 32-bit shift implicitly converted to 64 bits" warning. NFCI. (detail)
    by rksimon
  4. Don't update NoTrappingFPMath and FPDenormalMode in resetTargetOptions

    We'd like to remove this whole function, because these are properties of
    functions, not the target as a whole. These two are easy to remove
    because they are only used for emitting ARM build attributes, which
    expects them to represent the defaults for the whole module, not just
    the last function generated.

    This is needed to get correct build attributes when using IPRA on ARM,
    because IPRA causes resetTargetOptions to get called before
    ARMAsmPrinter::emitAttributes.

    Differential revision: https://reviews.llvm.org/D64929 (detail)
    by ostannard
Revision: 366290
Changes
  1. Revert "Revert r366458, r366467 and r366468"

    This reverts commit 9c377105da0be7c2c9a3c70035ce674c71b846af.

    [clangd][BackgroundIndexLoader] Directly store DependentTU while loading shard

    Summary:
    We were deferring the population of DependentTU field in LoadedShard
    until BackgroundIndexLoader was consumed. This actually triggers a use after
    free since the shards FileToTU was pointing at could've been moved while
    consuming the Loader.

    Reviewers: sammccall

    Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64980 (detail)
    by kadircet

Started by upstream project relay-test-suite-verify-machineinstrs build number 5605
originally caused by:

This run spent:

  • 5 sec waiting;
  • 7 min 50 sec build duration;
  • 7 min 55 sec total from scheduled to completion.