[PATCH] D34161: [AArch64] Add ARMv8.2-A FP16 vector intrinsics - Continuation

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

Not sure if I am missing something, but the new regression tests ##arm-v8.2a-neon-intrinsics.c## and ##aarch64-neon-intrinsics.c## are failing for me (after first applying patch https://reviews.llvm.org/D32511 and then this one here in https://reviews.llvm.org/D34161).  For example, the very first test in ##arm-v8.2a-neon-intrinsics.c##  checks that the function argument is ##<4 x half> %a##, but ##<4 x half> %1## is generated. With anything other than ##-O0## you will get the ##%a## (and then it becomes a tailcall).


https://reviews.llvm.org/D34161