[llvm-commits] [PATCH] ARM: Add vaddl/vsubl intrinsics

[Renato Golin]

On 20 December 2012 17:53, Bob Wilson <bob.wilson at apple.com> wrote:
> Sorry that I'm a bit behind in reviewing patches, but I don't think this is the right way to go.  We intentionally omitted those intrinsics because there is no need for them.  We can already represent those operations as combinations with vmovl.  Look at clang's <arm_neon.h> to see how they are implemented.

Hi Bob,

These are good points. It's been a while since I generated NEON
instructions from IR and I quite forgot that most extensions work by
pattern matching. I have no objection on reverting the patch, it's
quite non-invasive and doesn't change anything.

Do we have tests for those patterns? If not, we should.

Pete,

Can you generate those instructions as Bob described? I suggest we
revert the patch now and you try to generate them as patterns, and
create some tests to make sure they are correctly generated and don't
regress in the future.

--
cheers,
--renato

http://systemcall.org/