This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add TU variants to VPseudoBinaryM. NFC
AbandonedPublic

Authored by luke on Jun 15 2023, 4:23 AM.

Details

Summary

The aim is to use these in https://reviews.llvm.org/D152963 so that we
can preserve the passthru operand whilst also emitting the unmasked
variant. Like the mask variant, they are marked with forceTailAgnostic.

Diff Detail

Event Timeline

luke created this revision.Jun 15 2023, 4:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 15 2023, 4:23 AM
luke requested review of this revision.Jun 15 2023, 4:23 AM

Is BinaryM only used by compares? The name is not super helpful.

luke added a comment.Jun 23 2023, 7:36 AM

Is BinaryM only used by compares? The name is not super helpful.

Yeah, I think so. Should I rename it to VPsuedoVCMPM to match VPseudoVCMPM_VV_VF et al?

luke abandoned this revision.Aug 2 2023, 8:38 AM

After the pseudo refactoring this may no longer be relevant