[llvm-commits] Patch to add support for WidenVecRes_MERGE_VALUES

Eli Friedman eli.friedman at gmail.com
Fri Aug 26 19:02:31 PDT 2011


On Fri, Aug 26, 2011 at 6:34 PM, Villmow, Micah <Micah.Villmow at amd.com> wrote:
> This is a patch to fix LLVM bug10736 which occurs with our AMDIL backend.
> I've also added a test case, but since the AMDIL backend isn't in the tree yet, I don't know how much good it will do.
> Please review.

It looks correct, but it doesn't seem very general... there should
really just be a generic DecomposeMERGE_VALUES usable from every
handler that just has a single "ReplaceValueWith(SDValue(N, i),
SDValue(N->getOperand(i)));" loop.  (Long-term, we really should just
get rid of MERGE_VALUES, but it's a substantial amount of work.)

-Eli



More information about the llvm-commits mailing list