[PATCH] D42993: [AArch64] Fixes for ARMv8.2-A FP16 scalar intrinsic

Sjoerd Meijer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 7 12:48:56 PST 2018


SjoerdMeijer added a comment.

Thanks for fixing this. Looks very reasonable to me.

Question about the failures: I am now wondering if this means we were and still 
are missing tests?

Nit: for future reviews, I think it is better to split patches up if they are commits to 
different repos.


https://reviews.llvm.org/D42993





More information about the cfe-commits mailing list