This patch enables the use of the experimental reduction intrinsics from D30086 for AArch64.
The existing idiom recognition for the shufflevector sequence is removed and tests updated to use the new form.
Differential D32247
Switch AArch64 to use reduction intrinsics aemerson on Apr 19 2017, 2:47 PM. Authored by
Details This patch enables the use of the experimental reduction intrinsics from D30086 for AArch64. The existing idiom recognition for the shufflevector sequence is removed and tests updated to use the new form.
Diff Detail
Event TimelineComment Actions As soon as the dependency patch is approved, this one looks ok. Did you submit the patch that will make the MC layer recognise these patterns into the actual instructions?
Comment Actions No changes are needed on the MC side. The same target-specific reduction DAG nodes (e.g. AArch64ISD::UADDV) should be created and from then on everything should work as before.
Comment Actions New patch, rebased on latest ToT and using the different API implemented in the previous patch in D30086. Marking this as accepted based on Renato's earlier approval, I'm just updating this review for future reference. |