This is an archive of the discontinued LLVM Phabricator instance.

[ARM64EC] Add arm64ec for getArchName
ClosedPublic

Authored by bcl5980 on Sep 27 2022, 9:00 PM.

Details

Summary

Followup D125412, return the correct arch name for Arm64EC

Diff Detail

Event Timeline

bcl5980 created this revision.Sep 27 2022, 9:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 27 2022, 9:00 PM
bcl5980 requested review of this revision.Sep 27 2022, 9:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 27 2022, 9:00 PM
mstorsjo accepted this revision.Sep 28 2022, 3:17 AM

LGTM

Is there any testcase that could be added (e.g. for unittests for the Triple class) where this change makes a difference?

This revision is now accepted and ready to land.Sep 28 2022, 3:17 AM
bcl5980 updated this revision to Diff 463580.Sep 28 2022, 8:55 AM

correct test

efriedma accepted this revision.Sep 28 2022, 11:33 AM
mstorsjo accepted this revision.Sep 28 2022, 11:47 AM

LGTM, thanks! This test does indeed now test what I'd expect for this change.

This revision was automatically updated to reflect the committed changes.