This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Update cpu_specific test to use a newer CPU
ClosedPublic

Authored by FreddyYe on May 25 2023, 11:54 PM.

Diff Detail

Event Timeline

FreddyYe created this revision.May 25 2023, 11:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 25 2023, 11:54 PM
FreddyYe requested review of this revision.May 25 2023, 11:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 25 2023, 11:54 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
This revision is now accepted and ready to land.May 29 2023, 12:08 AM

I don't really see the justification here? Why do this change? If the intent is to just test a newer architecture, we can add tests for that, not change existing ones.

I don't really see the justification here? Why do this change? If the intent is to just test a newer architecture, we can add tests for that, not change existing ones.

KNL is deprecated and it is better to remove KNL related code in clang/llvm.

I don't really see the justification here? Why do this change? If the intent is to just test a newer architecture, we can add tests for that, not change existing ones.

KNL is deprecated and it is better to remove KNL related code in clang/llvm.

IS it deprecated in Clang? I see an RFC proposing it, but I don't see acceptance of that yet.

This patch is mainly for another WIP patch https://reviews.llvm.org/D151696, while I realize maybe it's too aggressive to support too many CPUs with -march, -mcpu, ... I'll reimplement the WIP patch to try to make the supports range same as before. Close this first.

FreddyYe closed this revision.Jun 13 2023, 5:45 PM