This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add missing plumbing and tests for zfa
ClosedPublic

Authored by reames on Feb 17 2023, 11:23 AM.

Details

Summary

Experimental support for the zfa extension was recently added in https://reviews.llvm.org/D141984. A couple of the normal test changes and clang plumbing got missed in that change. This commit updates the usual suspects.

Diff Detail

Event Timeline

reames created this revision.Feb 17 2023, 11:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 17 2023, 11:23 AM
reames requested review of this revision.Feb 17 2023, 11:23 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 17 2023, 11:23 AM
craig.topper added inline comments.Feb 17 2023, 11:31 AM
llvm/lib/Support/RISCVISAInfo.cpp
853

I think zfa should imply F here?

reames updated this revision to Diff 498512.Feb 17 2023, 2:53 PM

Address review comment

This revision is now accepted and ready to land.Feb 17 2023, 3:16 PM
This revision was automatically updated to reflect the committed changes.