[PATCH] D28135: SelectionDAG: Fix in legalization of UMAX/SMAX/UMIN/SMIN. Solves PR31486.

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 28 05:17:02 PST 2016


uabelho created this revision.
uabelho added reviewers: bogner, jvesely.
uabelho added a subscriber: llvm-commits.
Herald added a reviewer: tstellarAMD.
Herald added a subscriber: nhaehnle.

Originally

i64 = umax t8, Constant:i64<4>

was expanded into

i32,i32 = umax Constant:i32<0>, Constant:i32<0>
 i32,i32 = umax t7, Constant:i32<4>

Now instead the two produced umax:es return i32 instead of i32, i32.


https://reviews.llvm.org/D28135

Files:
  lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
  test/CodeGen/AMDGPU/legalize-umax-bug.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28135.82589.patch
Type: text/x-patch
Size: 3835 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161228/fa8d76d1/attachment.bin>


More information about the llvm-commits mailing list