This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Change naming of vector pseudos with scalar FP operand.
ClosedPublic

Authored by craig.topper on Aug 11 2023, 12:20 PM.

Details

Summary

We need a pseudo for each scalar FP register class. Previously
we distinquished the pseudos by naming them with F16, F32, F64, or
BF16 in place of the F in the normal instruction name.

Because these strings can appear in other parts of the name we had
to do things like matching "_VBF16" to "_VF".

This patch replaces the F16, F32, F64 strings with FPR16, FPR32, and
FPR64. We also use FPR16 for BF16 since that is the scalar register
class for bf16.

Since the FPR16/32/64 string does not anywhere else in the pseudo
names, we can use this to simplify the string replacements. This
also allows us to simplify some BF16 related code.

Diff Detail

Event Timeline

craig.topper created this revision.Aug 11 2023, 12:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 12:20 PM
craig.topper requested review of this revision.Aug 11 2023, 12:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 12:20 PM
Herald added subscribers: eopXD, MaskRay. · View Herald Transcript
wangpc accepted this revision.Aug 12 2023, 9:28 AM

LGTM.

This revision is now accepted and ready to land.Aug 12 2023, 9:28 AM