This is an archive of the discontinued LLVM Phabricator instance.

[10/15][Clang][RISCV][NFC] Don't need to check for `MaskPolicy` in `isTAPolicy` and `isTUPolicy`
ClosedPublic

Authored by eopXD on Jan 14 2023, 7:56 AM.

Details

Summary

Caller of the two utilities is always companied with a predicate to
check for !IsMask, so we don't need to check for the mask policy
here.

This also removes dependency to Policy::PolicyType::Omit, which will
be removed.

This is the 10th commit of a patch-set that aims to change the default policy
for RVV intrinsics from TAMU to TAMA.

Please refer to the cover letter in the 1st commit (D141573) for an
overview.

Diff Detail

Event Timeline

eopXD created this revision.Jan 14 2023, 7:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 14 2023, 7:56 AM
eopXD requested review of this revision.Jan 14 2023, 7:56 AM
eopXD updated this revision to Diff 489297.Jan 14 2023, 11:18 AM

Rebase upon latest main.

eopXD updated this revision to Diff 489305.Jan 14 2023, 11:26 AM

Bump CI.

eopXD retitled this revision from [WIP][10/N][Clang][RISCV][NFC] Don't need to check for `MaskPolicy` in `isTAPolicy` and `isTUPolicy` to [10/N][Clang][RISCV][NFC] Don't need to check for `MaskPolicy` in `isTAPolicy` and `isTUPolicy`.Jan 15 2023, 7:52 AM
eopXD edited the summary of this revision. (Show Details)
eopXD retitled this revision from [10/N][Clang][RISCV][NFC] Don't need to check for `MaskPolicy` in `isTAPolicy` and `isTUPolicy` to [10/15][Clang][RISCV][NFC] Don't need to check for `MaskPolicy` in `isTAPolicy` and `isTUPolicy`.
This revision is now accepted and ready to land.Jan 16 2023, 11:32 PM
This revision was landed with ongoing or failed builds.Jan 24 2023, 7:12 AM
This revision was automatically updated to reflect the committed changes.