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

Renato Golin rengolin at systemcall.org
Thu Dec 20 11:06:30 PST 2012


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/




More information about the llvm-commits mailing list