FP reductions no longer use these intrinsics since D78723.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
I was thinking more along the lines of "land the entire IntrinsicsAArch64.td separately", not just deleting the dead intrinsic patterns. Maybe there's something I'm missing?
Comment Actions
I thought you might have meant that, sorry I should have clarified. To be honest I'm struggling to find a reason for committing the IntrinsicsAArch64.td changes separately, am I missing something?
Comment Actions
LGTM
It makes review harder when I have to read a patch and spend extra time to figure out whether something that looks like an NFC change at first glance is actually supposed to be one.