Started 17 days ago
Took 4 hr 47 min on green-dragon-20

Success Build #18175 (Jun 9, 2019 11:02:22 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 362911
  • http://llvm.org/svn/llvm-project/cfe/trunk : 362887
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 362859
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 362745
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 362866
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 362811
Changes
  1. [InstCombine] foldICmpWithLowBitMaskedVal(): 'icmp sgt/sle': avoid miscompiles

    A precondition 'x != 0' was forgotten by me:
    https://rise4fun.com/Alive/JFNP
    https://rise4fun.com/Alive/jHvL

    These 4 folds with non-constants could be re-enabled,
    but for now let's go for the simplest solution.

    https://bugs.llvm.org/show_bug.cgi?id=42198 (detail/ViewSVN)
    by lebedevri
  2. [NFC][InstCombine] Revisit canonicalize-constant-low-bit-mask-and-icmp-s* tests in preparatio for PR42198.

    The `icmp sgt`/`icmp sle` variants are, too, miscompiles:
    https://rise4fun.com/Alive/JFNP
    https://rise4fun.com/Alive/jHvL
    A precondition 'x != 0' was forgotten by me.

    While ensuring test coverage for `-1`, also add test coverage
    for `0` mask. Mask `0` is allowed for all the folds,
    mask `-1` is allowed for all the folds with unsigned `icmp` pred.
    Constant mask `0` is missed though.

    https://bugs.llvm.org/show_bug.cgi?id=42198 (detail/ViewSVN)
    by lebedevri
  3. [InstCombine] change canonicalization to fabs() to use FMF on fneg

    This isn't the ideal fix (use FMF on the select), but it's still an
    improvement until we have better FMF propagation to selects and other
    FP math operators.

    I don't think there's much risk of regression from this change by
    not including the FMF on the fcmp any more. The nsz/nnan FMF
    should be the same on the fcmp and the fneg (fsub) because they
    have the same operand.

    This works around the most glaring FMF logical inconsistency cited
    in PR38086:
    https://bugs.llvm.org/show_bug.cgi?id=38086 (detail/ViewSVN)
    by spatel
  4. [NFC] Adjust test for D63004 (detail/ViewSVN)
    by xbolva00
  5. [NFC] Added test from PR19758 (detail/ViewSVN)
    by xbolva00
  6. [NFC] Added test from PR42084 for D63058 (detail/ViewSVN)
    by xbolva00
  7. [InstCombine] Add tests for usub.sat(x,y)+y etc; NFC

    For PR42178. (detail/ViewSVN)
    by nikic
  8. [InstSimplify] reduce code duplication for fcmp folds; NFC (detail/ViewSVN)
    by spatel
  9. [InstSimplify] enhance fcmp fold with never-nan operand

    This is another step towards correcting our usage of fast-math-flags when applied on an fcmp.
    In this case, we are checking for 'nnan' on the fcmp itself rather than the operand of
    the fcmp. But I'm leaving that clause in until we're more confident that we can stop
    relying on fcmp's FMF.

    By using the more general "isKnownNeverNaN()", we gain a simplification shown on the
    tests with 'uitofp' regardless of the FMF on the fcmp (uitofp never produces a NaN).
    On the tests with 'fabs', we are now relying on the FMF for the call fabs instruction
    in addition to the FMF on the fcmp.

    This is a continuation of D62979 / rL362879. (detail/ViewSVN)
    by spatel
  10. [InstSimplify] add tests for fcmp with known-never-nan operands; NFC

    Opposite predicate for rL362742 / rL362879 / D62979 (detail/ViewSVN)
    by spatel

Started by upstream project clang-stage2-Rthinlto_relay build number 1581
originally caused by:

This run spent:

  • 5 ms waiting;
  • 4 hr 47 min build duration;
  • 4 hr 47 min total from scheduled to completion.
Cobol Warnings: 0 warnings.
  • No warnings since build 10,378.
  • New zero warnings highscore: no warnings for 375 days!
Test Result (no failures)