[llvm-commits] Patch to add support for WidenVecRes_MERGE_VALUES

[Villmow, Micah]

> -----Original Message-----
> From: Eli Friedman [mailto:eli.friedman at gmail.com]
> Sent: Friday, August 26, 2011 7:03 PM
> To: Villmow, Micah
> Cc: llvm-commits
> Subject: Re: [llvm-commits] Patch to add support for
> WidenVecRes_MERGE_VALUES
> 
> 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
[Villmow, Micah] Here is an updated patch. This version merges the common
code between SplitRes_MERGE_VALUES and WidenVecRes_MERGE_VALUES into a
seperate function and has them call this function and then modify the
resulting value.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: AMDIL_Bug_10736-2.patch
Type: application/octet-stream
Size: 5099 bytes
Desc: AMDIL_Bug_10736-2.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110829/fd01e6a8/attachment.obj>