Skip to content

Fix test/Misc/target-cpu.swift #81137

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix test/Misc/target-cpu.swift #81137

wants to merge 2 commits into from

Conversation

MaxDesiatov
Copy link
Contributor

@MaxDesiatov MaxDesiatov commented Apr 28, 2025

It expected apple-a7, while the new value is apple-s4 for WATCHTARGETCPU2.

rdar://150074485

It expected `apple-a7`, while the new value is `apple-a4` for `WATCHTARGETCPU2`.
@MaxDesiatov MaxDesiatov added the test suite Area: test suite label Apr 28, 2025
@MaxDesiatov MaxDesiatov requested a review from al45tair April 28, 2025 13:18
@MaxDesiatov
Copy link
Contributor Author

@swift-ci smoke test

@MaxDesiatov MaxDesiatov enabled auto-merge (squash) April 28, 2025 13:21
@@ -14,7 +14,7 @@
// WATCHTARGETCPU1: "-target-cpu" "cortex-a7"

// RUN: not %swift -typecheck -target arm64-apple-watchos2 -Xcc -### %s 2>&1 | %FileCheck -check-prefix=WATCHTARGETCPU2 %s
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be arm64_32-apple-watchos2

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test linux

@MaxDesiatov MaxDesiatov dismissed ahmedbougacha’s stale review April 29, 2025 12:50

Requested changes added to the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test suite Area: test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants