Page MenuHomePhabricator

[SVE] Lower fixed length vector fneg and fsqrt operations.
ClosedPublic

Authored by paulwalker-arm on Thu, Oct 1, 10:43 AM.

Diff Detail

Event Timeline

paulwalker-arm created this revision.Thu, Oct 1, 10:43 AM
paulwalker-arm requested review of this revision.Thu, Oct 1, 10:43 AM
cameron.mcinally accepted this revision.Thu, Oct 1, 3:09 PM
cameron.mcinally added a subscriber: arsenm.

LGTM

llvm/test/CodeGen/AArch64/sve-fp.ll
333

Nit: These changes should probably go in with a standalone patch, but seem harmless enough to be included with this one.

@arsenm Also, there's an ongoing project to stop FMUL(-1.0, X)->FNEG(X) transformations when denormals are present, so this is a good change.

This revision is now accepted and ready to land.Thu, Oct 1, 3:09 PM
llvm/test/CodeGen/AArch64/sve-fp.ll
333

Just noticed a typo here:

Also, there's an ongoing project to stop FMUL(-1.0, X)->FNEG(X) transformations when denormals are present, so this is a good change.

Should say "when denormals are flushed".

This revision was landed with ongoing or failed builds.Tue, Oct 6, 2:57 AM
This revision was automatically updated to reflect the committed changes.