This is an archive of the discontinued LLVM Phabricator instance.

[7/15][Clang][RISCV][NFC] Correct the default value for Policy to TAMU
ClosedPublic

Authored by eopXD on Jan 14 2023, 5:22 AM.

Details

Summary

The default value is set to Omit, but in fact the value is then
assigned in updateNamesAndPolicy, which is set to TUMU when masked and
TAMU when unmasked. This commit demonstrates so and further remove
another dependency of Omit.

This is the 7th 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, 5:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 14 2023, 5:22 AM
eopXD requested review of this revision.Jan 14 2023, 5:22 AM
eopXD updated this revision to Diff 489234.Jan 14 2023, 5:23 AM

Bump CI

eopXD edited the summary of this revision. (Show Details)Jan 14 2023, 5:24 AM
eopXD retitled this revision from [WIP][7/N][Clang][RISCV][NFC] Correct the default value for Policy to undisturbed to [WIP][7/N][Clang][RISCV][NFC] Correct the default value for Policy to `Undisturbed`.Jan 14 2023, 5:30 AM
eopXD retitled this revision from [WIP][7/N][Clang][RISCV][NFC] Correct the default value for Policy to `Undisturbed` to [WIP][7/N][Clang][RISCV][NFC] Correct the default value for Policy to TAMU.Jan 14 2023, 7:45 AM
eopXD updated this revision to Diff 489252.Jan 14 2023, 7:46 AM

Update code: default is TAMU.

eopXD updated this revision to Diff 489294.Jan 14 2023, 11:13 AM

Rebase upon latest main.

eopXD updated this revision to Diff 489302.Jan 14 2023, 11:25 AM

Bump CI.

eopXD retitled this revision from [WIP][7/N][Clang][RISCV][NFC] Correct the default value for Policy to TAMU to [7/15][Clang][RISCV][NFC] Correct the default value for Policy to TAMU.Jan 15 2023, 7:38 AM
eopXD edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Jan 16 2023, 11:29 PM
This revision was landed with ongoing or failed builds.Jan 24 2023, 1:21 AM
This revision was automatically updated to reflect the committed changes.