Started 5 yr 5 mo ago
Took 48 min on green-dragon-08

Build #350 (Aug 12, 2017 2:50:51 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 310784
  • http://llvm.org/svn/llvm-project/cfe/trunk : 310778
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 310769
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 303903
  • http://llvm.org/svn/llvm-project/zorg/trunk : 310746
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 310761
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 310783
Changes
  1. [X86] When handling addcarry intrinsic, create the flag result with the correct type so we don't crash if we use a memory instruction

    Summary:
    Previously we were creating the flag result with MVT::Other which is interpretted as a Chain node. If we used a memory form of the instruction we would end up with a copyToReg that consumed the chain result of the adcx instruction instead of the flag result.

    Pretty sure we should be using MVT::i32 here, that's what we do other places we create these node types.

    We should probably consider this for 5.0 as well.

    Reviewers: RKSimon, zvi, spatel

    Reviewed By: RKSimon

    Subscribers: llvm-commits

    Differential Revision: https://reviews.llvm.org/D36645 (detail)
    by ctopper

Started by timer

This run spent:

  • 39 min waiting;
  • 48 min build duration;
  • 1 hr 28 min total from scheduled to completion.
Test Result (no failures)