This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add more patterns for FNMADD
ClosedPublic

Authored by liaolucy on Jun 1 2022, 8:52 PM.

Details

Summary

D54205 handles fnmadd: -rs1 * rs2 - rs3
This patch add fnmadd: -(rs1 * rs2 + rs3) (the nsz flag on the FMA)

Diff Detail

Event Timeline

liaolucy created this revision.Jun 1 2022, 8:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 1 2022, 8:52 PM
liaolucy requested review of this revision.Jun 1 2022, 8:52 PM
craig.topper requested changes to this revision.EditedJun 1 2022, 8:57 PM

This patch is only valid with the nsz flag on the FMA.

See also https://reviews.llvm.org/D109525 and https://reviews.llvm.org/D90901

This revision now requires changes to proceed.Jun 1 2022, 8:57 PM
liaolucy updated this revision to Diff 433707.Jun 2 2022, 3:01 AM

Address @craig.topper's comment.

Add a PatFrag to check presence of nsz on the fma.

liaolucy edited the summary of this revision. (Show Details)Jun 2 2022, 3:02 AM
This revision is now accepted and ready to land.Jun 2 2022, 9:26 PM
This revision was landed with ongoing or failed builds.Jun 3 2022, 9:31 PM
This revision was automatically updated to reflect the committed changes.