[LLVMbugs] [Bug 13839] llvm.umul.with.overflow.i64 intrinsic gives incorrect result, erroneously returns true for overflow on x86-32

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=13839

Eric Christopher <echristo at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Eric Christopher <echristo at gmail.com> 2012-09-13 18:29:53 CDT ---
I think I fixed this with:


commit 70d9d71687c92d962c2da098e9418cdfcc8cbcee
Author: echristo <echristo at 91177308-0d34-0410-b5e6-96231b3b80d8>
Date:   Thu Sep 13 23:24:02 2012 +0000

    Fix both the test for zero and what we do if we have a zero for
    umulo legalization.

    Fixes PR13839

    git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@163856
91177308-0d34-0410-b5e6-96231b3b80d8

commit e260dc65f94a6f8514b0f8c737ae99cb8183f578
Author: echristo <echristo at 91177308-0d34-0410-b5e6-96231b3b80d8>
Date:   Thu Sep 13 23:23:58 2012 +0000

    Reformat, remove a couple unused variables and move some variables
    closer to where they're needed.

    git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@163855
91177308-0d34-0410-b5e6-96231b3b80d8

-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.