[LLVMbugs] [Bug 19094] ACLE intrinics get compile error after r203229 commit

[bugzilla-daemon at llvm.org]

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

Tim Northover <t.p.northover at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |t.p.northover at gmail.com
         Resolution|---                         |FIXED

--- Comment #1 from Tim Northover <t.p.northover at gmail.com> ---
This is a pair of bugs in AArch64's LowerCONCAT_VECTORS, exposed by slightly
different incoming code.

1. The NEON_VDUP optimisation was far too aggressive, assuming (I think) that
the input would always be BUILD_VECTOR. In this case it was interpreting the
intrinsic ID as the number being duplicated.

2. We were treating most unknown concats as legal (by returning Op rather than
SDValue). I think only concats of pairs of vectors are actually legal.

I've committed a patch fixes it as r203450, but it looks like the code could be
improved by some combining of CONCAT_VECTORS and BUILD_VECTOR handling in these
weird cases (the CONCAT is essentially a BUILD, just with a pseudovector input
rather than a real vector).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20140310/e7a8990b/attachment.html>