Hi James, Thanks for reviewing. (Tim as well)
Updated patch attached, have retested and confirmed it all still passes. > Do you need help committing? Yes please. -Graham From: James Molloy <[email protected]<mailto:[email protected]>> Date: Friday, 5 September 2014 13:35 To: Admin <[email protected]<mailto:[email protected]>> Cc: llvm cfe <[email protected]<mailto:[email protected]>> Subject: Re: [PATCH] AArch32 v8 NEON intrinsics for numeric max/min and directed rounding to integral Hi Graham, Thanks for working on this. The implementation looks fine to me, but in your testcase: +float32x2_t test_vrndx_f32(float32x2_t a) { + // CHECK-LABEL: test_vrndx_f32 + // CHECK-LABEL: call <2 x float> @llvm.arm.neon.vrintx.v2f32(<2 x float> %a) + return vrndx_f32(a); +} Only the first should be a CHECK-LABEL. The other should be a CHECK. CHECK-LABELs are only hints to FileCheck, used to help isolate individual testcases and prevent accidental crosstalk (matching one CHECK: line in another function accidentally). With those changed, LGTM. Do you need help committing? Cheers, James On 5 September 2014 12:12, Graham Hunter <[email protected]<mailto:[email protected]>> wrote: Hi, This patch adds support for the 32bit numeric max/min and directed round-to-integral NEON intrinsics that were added as part of v8, along with unit tests. -Graham _______________________________________________ cfe-commits mailing list [email protected]<mailto:[email protected]> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits -- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2557590 ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2548782
vrint_and_maxmin_intrinsics_v2.patch
Description: vrint_and_maxmin_intrinsics_v2.patch
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
